Skip to content

feat(dev): add release:verify and repo:compliance --new-packagist-token, reorganize all commands#8756

Merged
bshaffer merged 7 commits intomainfrom
dev-update-packagist-token
Dec 11, 2025
Merged

feat(dev): add release:verify and repo:compliance --new-packagist-token, reorganize all commands#8756
bshaffer merged 7 commits intomainfrom
dev-update-packagist-token

Conversation

@bshaffer
Copy link
Copy Markdown
Contributor

@bshaffer bshaffer commented Nov 21, 2025

  • release:verify: Last release we had some packages fail to be updated. This adds a script to run after our release, so that we can be notified if releases do not show up in packagist.
  • repo-info --new-packagist-token: I believe the releases failed because the packagist tokens were rotated. The --update-packagist-token flag on the repo-info flag allows us to update the token across all repos at once
  • fixes bugs in repo-info command, renamed to repo:compliance, better output for repo:compliance
  • Reorganize all commands to use format component:update instead of component-update (for example), so that commands are organized better:
$ ./google-cloud
Available commands:
  completion                      Dump the shell completion script
  docfx                           Generate DocFX yaml from a phpdoc strucutre.xml
  help                            Display help for a command
  list                            List commands
 component
  component:add-version           Add a new version to an existing Component
  component:info                  [info] list info of a component or the whole library
  component:new                   Add a new Component
  component:update:deps           update a dependency across all components
  component:update:gencode        Update one or all components using Owlbot
  component:update:readme-sample  Add a sample to a component
 release
  release:info                    list information for a google-cloud-php release
  release:verify                  Verifies the package version from packagist.
 repo
  repo:compliance                 ensure all github repositories meet compliance
  repo:split                      [split] Split subtree and push to various remotes.

TODO

  • verify this worked on next release (it didn't work)
  • add verification step to ensure new versions were added to packagist

@bshaffer bshaffer changed the title feat(dev): add update-packagist-token option feat(dev): add verify-release and update-packagist-token Dec 1, 2025
@bshaffer bshaffer marked this pull request as ready for review December 4, 2025 21:13
@bshaffer bshaffer requested review from a team December 4, 2025 21:13
@bshaffer bshaffer added the next release PRs to be included in the next release label Dec 6, 2025
@bshaffer bshaffer changed the title feat(dev): add verify-release and update-packagist-token feat(dev): add release:verify and repo:compliance --new-packagist-token, reorganize all commands Dec 6, 2025
Comment thread dev/README.md
| Command | Description |
| ------------------- | --------------------------- |
| `sh/static-analysis`| Run phpstan static ananlysis|
| `sh/style-fix` | Run phpcs style check |
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I would love to make a command that locally can fix the style <3

Copy link
Copy Markdown
Contributor Author

@bshaffer bshaffer Dec 11, 2025

Choose a reason for hiding this comment

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

We have a composer style-fix command in the project root, although it probably needs to be cleaned up

@bshaffer bshaffer merged commit 53e704b into main Dec 11, 2025
48 checks passed
@bshaffer bshaffer deleted the dev-update-packagist-token branch December 11, 2025 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

next release PRs to be included in the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants