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 dependencies for Clojure and linting tools #160

Closed
wants to merge 4 commits into from
Closed

Conversation

SmetDenis
Copy link
Owner

@SmetDenis SmetDenis commented Feb 10, 2024

The lein-kibit plugin has been removed and replaced with splint linter. Clojure has also been updated to version 1.11.1. Corresponding changes have been made to the testing and linting tasks in bb.edn to accommodate the updated configurations.

Denis Smet added 3 commits February 10, 2024 19:18
This commit adds kibit - a static code analyzer - to the project's plugins in project.clj file. It also includes a new task in GitHub workflows for running kibit on the codebase, ensuring existing and new code complies with suggested style principles.
This commit replaces the standard equality checks in test assertions with more idiomatic Clojure testing functions. Changes include using (true?) and (false?) functions to check for boolean values and the (zero?) function for checking zero. This enhances readability and conforms to Clojure's idiomatic practices.
The lein-kibit plugin has been removed and replaced with splint linter. Clojure has also been updated to version 1.11.1. Corresponding changes have been made to the testing and linting tasks in bb.edn to accommodate the updated configurations.
@SmetDenis SmetDenis changed the title Add kibit to project and GitHub actions Update dependencies for Clojure and linting tools Feb 10, 2024
@SmetDenis SmetDenis closed this Feb 10, 2024
@SmetDenis SmetDenis deleted the kibit-3 branch February 10, 2024 17:46
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.

1 participant