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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixup errorcodes workflow #1313

Merged
merged 1 commit into from
Apr 3, 2024
Merged

fixup errorcodes workflow #1313

merged 1 commit into from
Apr 3, 2024

Conversation

phryneas
Copy link
Member

@phryneas phryneas commented Apr 3, 2024

.md is not a good filename for a changeset 馃槄

@phryneas phryneas requested a review from a team as a code owner April 3, 2024 12:36
@@ -21,7 +21,7 @@ jobs:
echo "No new error codes were added."
exit 0
fi
cat <<EOF > ../.changeset/`md5 < errorcodes.json`.md
cat <<EOF > ../.changeset/errorcodes.md
Copy link
Member Author

Choose a reason for hiding this comment

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

I believe it should be fine to always use the same name here, right @alessbell?

Copy link
Member

Choose a reason for hiding this comment

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

Generally yes if we can assume there would only ever be one PR with an errorcodes.md open at a time, but if you ever introduce a prerelease workflow I'd change this so the file name is unique.

Copy link
Member Author

Choose a reason for hiding this comment

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

Then let's revisit this if we ever get to the point of doing prereleases with the devtools ^^

Copy link
Member Author

Choose a reason for hiding this comment

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

As for the "ever only one" part - since they will all have the same message, it's probably even better that we don't repeat that over and over in the case of multiple PRs - it's not really important for the release message to know if error messages were updated once or three times between releases ^^

@phryneas phryneas merged commit 84fb904 into main Apr 3, 2024
7 checks passed
@phryneas phryneas deleted the pr/errorcodes-workflow branch April 3, 2024 13:11
@github-actions github-actions bot mentioned this pull request Apr 3, 2024
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.

None yet

2 participants