Repository navigation
docs: correct dprint bump and release bypass wording - #966
Merged
Merged
Conversation
dprint-update.yml said it bumps plugins to latest; dprint honours the 7-day cooldown in .npmrc, so a bump can stop short or skip a plugin. RELEASING.md named an Actions bypass for main only; the release flow also pushes to dev and a v* tag. Closes #958, closes #959 Co-Authored-By: Claude Fable 5.2 <[email protected]> Claude-Session: https://claude.ai/code/session_014tzYxya6Umczq4EBPTYkX4
Co-Authored-By: Claude Fable 5.2 <[email protected]> Claude-Session: https://claude.ai/code/session_014tzYxya6Umczq4EBPTYkX4
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.
Wording only; no behaviour changes.
dprint plugin bump (#958)
.github/workflows/dprint-update.ymlsaid in three places that it bumps every plugin to its latest version. dprint readsmin-release-age=7from.npmrc, so each plugin moves to its newest version that is at least 7 days old, and a bump can stop short of the latest release or skip a plugin. That made #957 look like a broken partial bump.Confirmed by running
dprint config update -yon the pre-#957dprint.jsonin a scratch directory, with and without the.npmrc:Without the
.npmrc, all four plugins went to their latest versions.The header comment, the step comment, the "nothing to do" log line and the PR body the workflow writes now say this.
Release settings (#959)
docs/RELEASING.mdnamed an Actions bypass for a ruleset onmainonly.auto-release.ymlalso pushes avX.Y.Ztag (bumpjob) and the merge ofmainback intodev(sync-devjob), so the bullet now coversmain,devandv*tags and names the job that fails without each.Working agreement
CLAUDE.mdgains a rule from the maintainer: an agent squash-merges its own PRs intodevonce they are green, and merges thedev→mainrelease PR only after the maintainer has approved that release.Checks
npx dprint check --list-differentpasses and the workflow file parses as YAML.actionlintwas not available locally; theLint (workflows)job in CI covers it.Closes #958, closes #959
Generated by Claude Code