Skip to content

update version to 0.5.0 #46

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
Sep 14, 2023
Merged

Conversation

philolo1
Copy link
Contributor

@philolo1 philolo1 commented Sep 4, 2023

No description provided.

@sylvestre
Copy link
Contributor

I think cargo.lock needs to be updated too, no?

@philolo1 philolo1 force-pushed the feature/update-version branch from 5ecf0b3 to f2d9b95 Compare September 4, 2023 07:05
@philolo1
Copy link
Contributor Author

philolo1 commented Sep 4, 2023

@sylvestre updated.

@sylvestre
Copy link
Contributor

i wish your update of cargo.lock didn't update many dependencies in this PR

@philolo1
Copy link
Contributor Author

philolo1 commented Sep 4, 2023

@sylvestre i just ran "cargo update", is there another way? My guess is that chrono has quite some dependency changes. I don't remember adding any library.

@tertsdiepraam
Copy link
Member

@philolo1 Just cargo test or something like that should fix Cargo.lock right? Idk, maybe I'm confused. cargo update is specifically for updating all dependencies to the highest versions in Cargo.toml

@philolo1
Copy link
Contributor Author

philolo1 commented Sep 4, 2023

@tertsdiepraam @sylvestre if i run cargo test there is no change to cargo lock file. I don't think the cargo lock file has any reference to the cargo version. I can set it back to not changing the lock file. Unfortunatly i don't have to much experience with rust release system.

@@ -135,35 +138,35 @@ checksum = "dd8b5dd2ae5ed71462c540258bedcb51965123ad7e7ccf4b9a8cafaa4a63576d"

[[package]]
name = "parse_datetime"
version = "0.4.0"
Copy link
Member

Choose a reason for hiding this comment

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

Well, this should be in there at least. Is that not changed either?

Copy link
Member

Choose a reason for hiding this comment

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

Oh and the fuzz/Cargo.lock should be updated too I guess.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cakebaker
Copy link
Collaborator

Maybe it's cargo update -p <package-name> you are looking for?

@philolo1 philolo1 force-pushed the feature/update-version branch from f2d9b95 to 9a2c22c Compare September 4, 2023 08:49
update fuzz to version to 0.2.0
@philolo1 philolo1 force-pushed the feature/update-version branch from 9a2c22c to 1e43cc9 Compare September 4, 2023 08:51
@philolo1
Copy link
Contributor Author

philolo1 commented Sep 4, 2023

@cakebaker thanks.
@tertsdiepraam @sylvestre thanks for the comments, i think i now updated only parse_datetime and fuzz.

@philolo1
Copy link
Contributor Author

@tertsdiepraam @sylvestre any updates, i would like to close the pr's i have here.

@sylvestre
Copy link
Contributor

oups, sorry :(

@sylvestre sylvestre merged commit 02e5c22 into uutils:main Sep 14, 2023
@sylvestre
Copy link
Contributor

ok to publish on crates.io ?

@philolo1
Copy link
Contributor Author

philolo1 commented Sep 14, 2023 via email

@sylvestre
Copy link
Contributor

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.

4 participants