Ignore missing deps file in `neil new` - Githubissues

babashka / neil

A CLI to add common aliases and features to deps.edn-based projects

MIT License

377 stars 27 forks source link

Ignore missing deps file in `neil new` #242

Closed bobisageek closed 1 month ago

bobisageek commented 1 month ago

Closes #241

Change Summary

when neil new output doesn't contain a deps file to update, do nothing (instead of throwing)
update CI task to have valid task names
update "latest version" test to expect clojure 1.12.0
add tests for assoc-project-meta!

Change Detail(s) by file

bb.edn - the ci task appears to call outdated(?) task names, so these are changed to run all the tasks describe in the Actions workflows
neil - updated just based on the run of gen-script - includes some changes that appear not have not been generated from prior commits
new.clj - only call assoc-project-meta! if the deps file exists in the directory created by neil new
dep_add_test.clj - newer version of clojure expected in test

Notes

might want to add the other test namespaces to test_runner, or maybe switch to using cognitect test runner... I didn't want to increase the scope of this change too much
[x] This PR corresponds to an issue with a clear problem statement.
[x] I have updated the CHANGELOG.md file with a description of the addressed issue.
[x] I have considered whether I should add more tests covering the code I've changed.

borkdude commented 1 month ago

Good for merge now it seems?

bobisageek commented 1 month ago

Yep - I've got everything in that I'm aiming for.