[CHORE] Break out attribution.mdx content into separate file#385
Merged
Conversation
Copilot started reviewing on behalf of
Adam Lastowka (Rachmanin0xFF)
May 14, 2026 20:35
View session
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a stable marker in the attribution documentation so an external release automation (release DAG) can deterministically locate where to inject generated attribution content when opening release PRs.
Changes:
- Inserted an HTML comment marker (
GENERATED_ATTRIBUTION_START) immediately after the attribution screenshot to serve as an insertion point for automated content generation.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
Francisco Jiménez (jjcfrancisco)
approved these changes
May 15, 2026
Adam Lastowka (Rachmanin0xFF)
force-pushed
the
attribution-marker
branch
2 times, most recently
from
b214385 to
9b753ec
Compare
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
Adam Lastowka (Rachmanin0xFF)
changed the title
attribution.mdx content into seperate file
Break out the generated attribution content from attribution.mdx into _generated_attribution.mdx, imported as a component. The release DAG can overwrite the data file without touching the parent page's formatting. Signed-off-by: Adam Lastowka <alastowka@overturemaps.org> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Adam Lastowka <adamlastowka@gmail.com>
Adam Lastowka (Rachmanin0xFF)
force-pushed
the
attribution-marker
branch
from
ae548a9 to
2f5fa66
Compare
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
Contributor
Author
|
staging deploy broken but the page looks fine locally |
Adam Lastowka (Rachmanin0xFF)
changed the title
attribution.mdx content into seperate fileattribution.mdx content into separate file
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.
Closes #386
Pull Request
Adds a comment so that the release DAG knows where to insert the attribution artifact contents when it opens a PR (see here)