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

Fix deprecation warning on Formatting.jl package #57

Merged
merged 3 commits into from
Sep 8, 2024

Conversation

pmc4
Copy link
Contributor

@pmc4 pmc4 commented Sep 7, 2024

When installing SpelledOut.jl in a new Julia environment, a deprecation warning by the Formatting.jl package is given.

As can be see in its repository, the package is deprecated and should be interchanged with the more modern Format.jl package from here.

This branch fixes that deprecation warning by changing both packages. The only imported function from Formatting.jl was format. The same syntax holds for the Format.jl so no further changes must be made. The branch passed all the tests locally.

pmc4 and others added 3 commits September 7, 2024 12:38
Because
* The package was deprecated and the new Format.jl package should be
used instead

This commit only changes one package with another and fixes the
deprecation warning.
Formatting.jl is deprecated (as of March, 2024), in favour of
JuliaString's Format.jl.  The API we are using to format numbers is
the same, and this dependency change does not change any of the
functionality for users of SpelledOut.jl.

This fix was implemented by @pmc4.

See merge request #57
@jakewilliami
Copy link
Owner

Thanks for this @pmc4! Great stuff—I didn't know Formatting.jl was deprecated (I wrote this a while ago). I know Scott (@ScottPJones) from the Humans of Julia Discord server, and Format.jl looks great. Just needed to do the same change in the documentation directory (there's probably a better way to do this without replicating the package dependencies in docs/).

@jakewilliami jakewilliami merged commit d12f348 into jakewilliami:master Sep 8, 2024
7 checks passed
@pmc4 pmc4 deleted the fix-deprecation branch September 8, 2024 08:15
@pmc4 pmc4 restored the fix-deprecation branch September 8, 2024 08:15
@pmc4
Copy link
Contributor Author

pmc4 commented Sep 8, 2024

No problem @jakewilliami ! I forgot about the docs, thanks for pointing that out. I'll bear that in mind for the next time. I just came across your package when solving one of the Project Euler problems.

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