Suppress timestamp-only generated docs PRs - #1167
Draft
danielporterda wants to merge 1 commit into
Draft
Conversation
Signed-off-by: danielporterda <daniel.porter@digitalasset.com>
danielporterda
temporarily deployed
to
cf-docs-generated-docs-merger
July 28, 2026 16:31 — with
GitHub Actions
Inactive
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What changed
_generatedmetadata block when the published dashboardversionsandrepositoriesare unchanged.Why
The dashboard generator previously preserved
generatedAtonly when every other provenance field was identical, while the PR orchestrator treated any source-file diff as sufficient to proceed toward publication. Incidental collection metadata could therefore make a no-value update look substantive.After this change, metadata-only collection runs return before rendered-tab generation and before
create_or_update_pull_requestis called. No timestamp-only PR is created. The existing stale-PR cleanup is only for removing a PR left open by the old behavior.Validation
direnv exec . python3 -m pytest tests/test_generate_network_component_versions.py tests/test_update_generated_reference_prs.py tests/test_summarize_version_changes.py -q(59 passed)direnv exec . ruff check scripts/generate_network_component_versions.py scripts/update_generated_reference_prs.py tests/test_generate_network_component_versions.py tests/test_update_generated_reference_prs.pygit diff --checknpm run generate:version-compatibility-dashboardplusnpm run generate:network-variable-tabsproduced no generated-output drift.python3 scripts/generate_all_reference_docs.py --dry-runlisted all ten reference wrappers successfully.Update generated docsrun 30378676808 withtargets=version-dashboardcompleted successfully, loggedSource unchanged for Update generated docs; skipping generation, and created noversion-dashboard/updatePR against this branch.