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

updating msrv #747

Merged
merged 2 commits into from Sep 18, 2023
Merged

updating msrv #747

merged 2 commits into from Sep 18, 2023

Conversation

sanbox-irl
Copy link
Member

  • this PR adds a formal MSRV to the repo (there's no reason we shouldn't use one)
  • bumps that MSRV to 1.70
  • moves rustfmt's check to run only on main
  • reduces clippy to only run on stable rust, not on MSRV AND makes it only run on main
  • reduces testing to ONLY the MSRV because of Rusts forwards compatibility concerns
  • moves all crates to edition 2021
  • moves us to formally use resolver = 2 in the workspace setting

- clippy runs on stable
- testing runs on msrv only
@sanbox-irl sanbox-irl requested a review from dbr September 17, 2023 18:11
@sanbox-irl sanbox-irl merged commit 4c80d0d into imgui-rs:main Sep 18, 2023
5 checks passed
@sanbox-irl sanbox-irl deleted the updating-msrv branch September 18, 2023 12:52
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.

None yet

1 participant