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

Intern defnk #56

Closed
wants to merge 14 commits into from
Closed

Intern defnk #56

wants to merge 14 commits into from

Conversation

ccfontes
Copy link
Owner

No description provided.

examples/bb-hiccup/handler.clj Outdated Show resolved Hide resolved
examples/bb-hiccup/handler.clj Outdated Show resolved Hide resolved
examples/bb-map-context/handler.clj Outdated Show resolved Hide resolved
examples/bb-map-context/handler.clj Outdated Show resolved Hide resolved
examples/bb-routes/handler.clj Outdated Show resolved Hide resolved
examples/bb-routes/handler.clj Outdated Show resolved Hide resolved
…unresolved-symbol exclusions

The commit updates the `.clj-kondo/config.edn` configuration file to exclude the `defnk` macro from the `unresolved-symbol` linter checks, in addition to the existing exclusions. This change is likely made to prevent false positives from the linter for symbols that are valid in the project's context but not recognized by clj-kondo by default.
feat(bb.edn): add prismatic/plumbing dependency to bb.edn

The specific dependency preparation step in the Dockerfile is removed to
simplify the build process. Instead, the dependency is now declared in
the bb.edn file, ensuring that all required dependencies are managed in
a single, centralized configuration file. This change improves the
maintainability of dependency management and streamlines the Docker build
process.
chore(template/bb/Dockerfile): update babashka image to 1.3.187-SNAPSHOT

The changes explain the use of a SNAPSHOT version of the babashka Docker image to access an unreleased feature (`defnk`) and prepare for a future upgrade to a stable release. The Dockerfile is updated accordingly to reflect this change.
…script

refactor(index.clj): remove direct intern of 'defnk', use 'interns' ns for side-effects
feat(interns.clj): add new 'interns' namespace to manage interning of symbols
BREAKING CHANGE: Removal of custom defnk macro may affect existing codebases that rely on this macro from interns.clj. Users should replace usages with equivalent core or third-party macros.
refactor(bb-map-context/handler.clj): use destructuring in handler function for better code readability
@ccfontes ccfontes closed this Dec 20, 2023
@ccfontes ccfontes deleted the intern-defnk branch December 20, 2023 19:10
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