Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tooling to programmatically update diagnostic-related files #6368

Merged
merged 2 commits into from
Jan 30, 2025

Conversation

parlough
Copy link
Member

@parlough parlough commented Jan 27, 2025

This mostly moves some tooling from the SDK and doesn't need to be too closely reviewed. It adds ./dash_site generate-diagnostics to update the src/_data/linter_rules.json and src/content/tools/diagnostic-messages.md files from sources in the SDK.

This is temporary and much of it will be expanded or replaced with logic in the static site generator to generate individual diagnostic message pages. Then it will be automatically ran every night, reducing the manual steps necessary. I'm proposing we land this so we can incrementally get to that point and enable simplifying the SDK implementation.

@dart-github-bot
Copy link
Collaborator

dart-github-bot commented Jan 27, 2025

Visit the preview URL for this PR (updated for commit caebe07):

https://dart-dev--pr6368-misc-update-diagnostics-tooling-6sc5mr1f.web.app

@parlough parlough changed the title Add tooling to programmatically generate diagnostic-related files Add tooling to programmatically update diagnostic-related files Jan 28, 2025
Copy link
Contributor

@MaryaBelanger MaryaBelanger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good in staging, thanks!

@parlough parlough merged commit 172e28f into main Jan 30, 2025
9 of 10 checks passed
@parlough parlough deleted the misc/update-diagnostics-tooling branch January 30, 2025 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants