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

Update env logger #49

Merged
merged 4 commits into from
Jun 30, 2022
Merged

Conversation

complexspaces
Copy link
Contributor

@complexspaces complexspaces commented Jul 1, 2021

This PR updates env-logger to version 0.9 since it's been out for a few months now (and 0.8 even longer).

The only upstream breaking change is the MSRV being increased to 1.41. However, while I don't see a MSRV for pretty_env_logger, its doubtful it will break any users as Debian stable ships a minimum of 1.41.1 and the majority of users are on newer compiler versions.

Closes #50

@djc
Copy link

djc commented Oct 6, 2021

@complexspaces maybe update this to env-logger 0.9? @seanmonstar ping?

@complexspaces
Copy link
Contributor Author

@djc Thanks for the ping! I've updated this branch to use0.9 instead.

@complexspaces
Copy link
Contributor Author

@seanmonstar Friendly ping on this PR, if you have the bandwidth available.

Copy link
Owner

@seanmonstar seanmonstar left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, and sorry for the delay.

Cargo.toml Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
@seanmonstar
Copy link
Owner

Oh noes, this repo is still configured with old CI... Sigh, need to update to GH Actions.

@praveenperera
Copy link
Contributor

@complexspaces @seanmonstar now that GitHub actions is setup can we rebase on master and merge this in?

@complexspaces
Copy link
Contributor Author

I've rebased onto the latest master, but CI will now need manually run.

@complexspaces
Copy link
Contributor Author

complexspaces commented Jun 10, 2022

@seanmonstar @praveenperera Sorry for the delay. I've ran rustfmt on this branch as CI is now requiring it and that was failing previously. This branch should be ready to merge.

@seanmonstar seanmonstar merged commit f9e35b6 into seanmonstar:master Jun 30, 2022
@complexspaces complexspaces deleted the update-env-logger branch June 30, 2022 16:45
@atouchet atouchet mentioned this pull request Aug 5, 2022
@jpmckinney
Copy link

Can you create a release that includes this PR?

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.

Please bump env_logger dependency to 0.9
5 participants