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

Chore/update dependencies #29

Merged
merged 5 commits into from
Jun 5, 2024

Conversation

365tuwe
Copy link
Contributor

@365tuwe 365tuwe commented May 28, 2024

Update project dependencies specified in Cargo.toml, cargo update to generate new Cargo.lock file

Main reasons:

Cargo.toml Outdated Show resolved Hide resolved
env_logger = "0.10"
fluent-templates = { version = "0.8", optional = true, default-features = false, features = ["tera"]}
env_logger = "0.11"
fluent-templates = { version = "0.9", optional = true, default-features = false, features = ["tera", "use-ignore"] }
Copy link
Owner

Choose a reason for hiding this comment

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

What is use-ignore for ?

Copy link
Contributor

Choose a reason for hiding this comment

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

There are two features, use-ignore and walkdir, one of which must be enabled. The default feature list includes use-ignore, so this is not a change except that by dropping default features this dependency now doesn't need the macros feature to be compiled. I assume that was the point of this change...which makes sense since they are not being used in this context.

@chevdor chevdor merged commit 6db2f6b into chevdor:master Jun 5, 2024
5 checks passed
@365tuwe
Copy link
Contributor Author

365tuwe commented Jun 5, 2024

Great to see that this got merged so fast and a new release is available. Unfortunately, someone else got tagged in the 0.4.0 release notes as contributor - is it possible to adjust this?

@chevdor
Copy link
Owner

chevdor commented Jun 5, 2024

Oh yes sure, don't worry about that and sorry for the automation errors. I will adjust.
Thanks for the 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.

3 participants