Skip to content

Sort targets in the buf_breaking rules #100

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

Merged

Conversation

dspencer12
Copy link
Contributor

Updates the generation of the buf_breaking rule to sort the targets for stable Bazel output. Updates breaking test cases to include multiple targets.

@dspencer12 dspencer12 marked this pull request as ready for review April 9, 2025 18:02
@dspencer12
Copy link
Contributor Author

Tests are failing due to actions/cache v2 deprecation; should be fixed by #101

Updates the generation of the `buf_breaking` rule to sort the
targets for stable Bazel output. Updates breaking test cases
to include multiple targets.
@dspencer12 dspencer12 force-pushed the stable-buf-breaking-targets branch from af97cfd to a131f0e Compare April 21, 2025 22:15
@srikrsna-buf srikrsna-buf merged commit 6631c04 into bufbuild:main Apr 24, 2025
4 checks passed
@dspencer12 dspencer12 deleted the stable-buf-breaking-targets branch April 24, 2025 16:40
@srikrsna-buf srikrsna-buf mentioned this pull request May 7, 2025
srikrsna-buf added a commit that referenced this pull request May 7, 2025
## What's Changed
* Use canonical_id in rctx.download and use the latest github release
version by @Strum355 in #62
* Fix running tests in Windows by @jchadwick-buf in
#79
* Update bzlmod example to be able to use toolchain by @sfroment in
#81
* Fix extensions.bzl to allow for multiple versions by @filippobrizzi in
#88
* Add repository_url to reproducible attributes by @styurin in
#94
* Sort targets in the buf_breaking rules by @dspencer12 in
#100
* Add `buf_format` rule by @srikrsna-buf in
#105

## New Contributors
* @Strum355 made their first contribution in
#62
* @jchadwick-buf made their first contribution in
#79
* @sfroment made their first contribution in
#81
* @filippobrizzi made their first contribution in
#88
* @dspencer12 made their first contribution in
#101
* @styurin made their first contribution in
#94

**Full Changelog**:
v0.3.0...v0.4.0
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.

2 participants