Skip to content

chore: refresh mise-php admin state snapshot - #25

Merged
loadinglucian merged 1 commit into
mainfrom
chore/refresh-admin-state
Aug 2, 2026
Merged

chore: refresh mise-php admin state snapshot#25
loadinglucian merged 1 commit into
mainfrom
chore/refresh-admin-state

Conversation

@loadinglucian

@loadinglucian loadinglucian commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Recaptures mise-php's GitHub configuration now that the autorelease environment rename and the MAINTENANCE_OWNER removal have landed.

What actually moved, and why:

  • The php-autorelease-publish environment's ID, node ID, protection rule ID and timestamps changed. GitHub environments are configuration rather than repository files, so the source rename could not move them. The live environment was still named php-maintenance-release; it was replaced with a fresh php-autorelease-publish carrying identical protection (can_admins_bypass: false, protected_branches: true, one branch policy rule). The old one was confirmed to hold zero deployments, secrets and variables before deletion.
  • The environment name shows no diff because the previous snapshot already read php-autorelease-publish while GitHub still had the old name. The file was recording the intended state; it now records the real one.
  • repository changed from Bigpixelrocket/mise-php to bigpixelrocket/mise-php. The script records the --repo argument verbatim rather than the canonical full_name it already fetches. The lowercase form is what docs/repository-settings.md documents, so future captures reproduce it.
  • The autorelease label moved to alphabetical position. The script stores label names in API order without sorting; the set of twelve names is unchanged.

MAINTENANCE_OWNER does not appear as a removal because the previous snapshot already listed only AUTORELEASE_OWNER.

No secret values are present. secretNames holds bare names only.

Summary by CodeRabbit

  • Documentation
    • Updated the administrative state snapshot to reflect the latest repository metadata, labels, identifiers, timestamps, and validation information.
    • No changes were made to application functionality or publicly exposed features.

@cursor

cursor Bot commented Aug 2, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_f5bd9491-a1de-43e1-b4b8-7d61c01418a2)

@coderabbitai

coderabbitai Bot commented Aug 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0b7ec1da-4203-4062-929d-1eb7abf46d78

📥 Commits

Reviewing files that changed from the base of the PR and between 26562c8 and e4b87e7.

📒 Files selected for processing (1)
  • docs/admin-state/mise-php-after.json

📝 Walkthrough

Walkthrough

The administrative-state snapshot updates environment metadata, adds the autorelease label, normalizes the repository value to lowercase, and records a new snapshot digest.

Changes

Administrative state snapshot

Layer / File(s) Summary
Refresh administrative state snapshot
docs/admin-state/mise-php-after.json
The snapshot updates timestamps and identifiers, adds the autorelease label, changes the repository value to lowercase, and updates the digest.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the snapshot changes but omits the required Summary, Verification, and Security section structure and verification results. Use the repository template headings and add verification results for tests, behavior, release handling, compatibility, checksum enforcement, and committed-file security.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the snapshot refresh and matches the pull request's primary change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/refresh-admin-state

Comment @coderabbitai help to get the list of available commands.

@loadinglucian
loadinglucian merged commit dd6e72e into main Aug 2, 2026
5 checks passed
@loadinglucian
loadinglucian deleted the chore/refresh-admin-state branch August 2, 2026 20:13
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.

1 participant