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 tools.deps to 0.18.1398. #245

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Conversation

r0man
Copy link
Contributor

@r0man r0man commented Jan 5, 2024

Tools deps shipped with a spec that had a typo in it [1]. If the specs namespace of tools.deps is loaded, and instrumentation is turned on for a project, then the typo will cause an error.

I have a project that depends on antq which pulls in this bad spec and I can't run tests with instrumentation turned on.

This change updates tools.deps to fix this issue.

[1] clojure/tools.deps@f71924e
[2] https://ask.clojure.org/index.php/13601/typo-in-clojure-tools-deps-specs-claspath-args

Tools deps shipped with a spec that had a typo in it [1]. If the specs
namespace of tools.deps is loaded, and instrumentation is turned on
for a project, then the typo will cause an error.

I have a project that depends on antq which pulls in this bad spec and
I can't run tests with instrumentation turned on.

This change updates tools.deps to fix this issue.

[1] clojure/tools.deps@f71924e
[2] https://ask.clojure.org/index.php/13601/typo-in-clojure-tools-deps-specs-claspath-args
@liquidz liquidz merged commit 286ade2 into liquidz:main Jan 5, 2024
1 check passed
@liquidz
Copy link
Owner

liquidz commented Jan 5, 2024

@r0man Thanks for your notice!

@liquidz
Copy link
Owner

liquidz commented Jan 5, 2024

@r0man Just released v2.8.1169 :)

@r0man
Copy link
Contributor Author

r0man commented Jan 5, 2024

@liquidz Thank you!

@r0man r0man deleted the update-tools-deps branch January 5, 2024 11:29
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