8000 Only check library against MSRV by djc · Pull Request #1302 · rustls/rustls · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Only check library against MSRV #1302

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
merged 1 commit into from
May 22, 2023
Merged

Only check library against MSRV #1302

merged 1 commit into from
May 22, 2023

Conversation

djc
Copy link
Member
@djc djc commented May 22, 2023

To avoid unnecessary MSRV bumps that are required for dev-dependencies or the examples project, only check the MSRV for the rustls library target.

We've recently started using this in Quinn, and I think it's a pretty nice improvement to avoid unnecessary MSRV bumps.

@codecov
Copy link
codecov bot commented May 22, 2023

Codecov Report

Merging #1302 (45fd557) into main (792045b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1302   +/-   ##
=======================================
  Coverage   95.55%   95.55%           
=======================================
  Files          60       60           
  Lines       14351    14351           
=======================================
  Hits        13713    13713           
  Misses        638      638           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member
@cpu cpu left a comment

Choose a reason for hiding this comment

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

This seems reasonable to me. Thanks @djc 👍

@djc djc merged commit c8e9e0b into main May 22, 2023
@djc djc deleted the lib-msrv branch May 22, 2023 13:36
@djc djc mentioned this pull request May 22, 2023
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
0