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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

semver_checks: Revert previous workaround #1051

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

Lorak-mmk
Copy link
Collaborator

The change in semver-checks turned out to be intentional and cargo install cargo-semver-checks --no-default-features is no longer expected to work.
The replacement command is cargo install cargo-semver-checks --no-default-features --features gix-reqwest.

See the release notes for v0.33.0: https://github.com/obi1kenobi/cargo-semver-checks/releases/tag/v0.33.0

Pre-review checklist

  • I have split my patch into logically separate commits.
  • All commit messages clearly explain what they change and why.
  • I added relevant tests for new features and bug fixes.
  • All commits compile, pass static checks and pass test.
  • PR description sums up the changes and reasons why they should be introduced.
  • I have provided docstrings for the public items that I want to introduce.
  • I have adjusted the documentation in ./docs/source/.
  • I added appropriate Fixes: annotations to PR description.

Copy link

github-actions bot commented Aug 1, 2024

cargo semver-checks found no API-breaking changes in this PR! 🎉🥳
Checked commit: e79becf

@wprzytula
Copy link
Collaborator

Are you going to open a PR to this branch in order to check that the Semver Checks are indeed working after the changes?

@Lorak-mmk
Copy link
Collaborator Author

Yes: Lorak-mmk#2
it passes, apart from removing label because those labels are not present on my fork.

Copy link
Collaborator

@wprzytula wprzytula left a comment

Choose a reason for hiding this comment

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

Please remove the mock checked branch and I'll merge.

The change in semver-checks was intentional and
`cargo install cargo-semver-checks --no-default-features`
is no longer expected to work. The replacement command is
`cargo install cargo-semver-checks --no-default-features --features gix-reqwest`.

See the release notes for v0.33.0: https://github.com/obi1kenobi/cargo-semver-checks/releases/tag/v0.33.0
@Lorak-mmk Lorak-mmk force-pushed the semver-checks-revert branch from 8400c32 to e79becf Compare August 1, 2024 09:00
@Lorak-mmk
Copy link
Collaborator Author

Done @wprzytula

@wprzytula wprzytula merged commit a707bfd into scylladb:main Aug 1, 2024
10 checks passed
wprzytula added a commit to wprzytula/scylla-rust-driver that referenced this pull request Aug 22, 2024
semver_checks: Revert previous workaround

(cherry picked from commit a707bfd)
@wprzytula wprzytula mentioned this pull request Aug 22, 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.

2 participants