Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Recent changes around the netlify part of the website build meant running the build locally didn't always generate all of the product docs.
I believe this was due to a timing issue around the copy_docs script running in parallel to 11ty starting up - a timing issue that was exposed by removing the netlify part of the build.
This PR modifies how the copy_docs/watch_docs scripts operate so that:
copy_docs
can be used in parallel to 11ty starting. This only copies things over when changes are made - it does not do an initial copy over.To achieve this, I've scrapped the
watch_docs
script that usednodemon
- it was a sledgehammer that would copy the full docs tree over on every change - leading 11ty to rebuild far more than it needed it.The new watch mode handles individual file changes and only copies/creates/deletes files that have changed.
This is working well locally - but will appreciate some local testing by others before merging.
I haven't (yet) touched the blueprint copy/watch scripts, but if this approach works well for core docs, I'll refactor those scripts to reuse what I've done here.