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

Moving to MSRV 1.36.0 #70

Merged
merged 1 commit into from May 8, 2021
Merged

Moving to MSRV 1.36.0 #70

merged 1 commit into from May 8, 2021

Conversation

dr-orlovsky
Copy link
Member

@dr-orlovsky dr-orlovsky commented May 7, 2021

This will allow to use crates in rust-bitcoin projects, which will require rustc 1.36 from the next major release (rust-bitcoin/rust-bitcoin#510 (comment))

NB: MSRV will change only when stringly_conversions and serde_str_helpers are not used. With them, MSRV will remain 1.41.1. This difference is already reflected in tests and README

@dr-orlovsky dr-orlovsky added the enhancement New feature or request label May 7, 2021
@dr-orlovsky dr-orlovsky added this to the v3.0 milestone May 7, 2021
@dr-orlovsky dr-orlovsky requested a review from Kixunil May 7, 2021 22:55
@dr-orlovsky dr-orlovsky added this to In progress in Software stack & toolchain via automation May 7, 2021
@codecov
Copy link

codecov bot commented May 7, 2021

Codecov Report

Merging #70 (5c40ade) into master (51178fb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #70   +/-   ##
======================================
  Coverage    63.9%   63.9%           
======================================
  Files          26      26           
  Lines        3347    3347           
======================================
  Hits         2138    2138           
  Misses       1209    1209           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 51178fb...5c40ade. Read the comment docs.

@dr-orlovsky dr-orlovsky force-pushed the develop branch 3 times, most recently from fd7c783 to 060a459 Compare May 8, 2021 00:33
@dr-orlovsky dr-orlovsky merged commit 5c40ade into master May 8, 2021
Software stack & toolchain automation moved this from In progress to Done May 8, 2021
Copy link
Contributor

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

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

Post-merge looks good :)

@dr-orlovsky
Copy link
Member Author

Very strange thing, I didn't merge this PR. Seems like I did that by occasion locally from command line :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants