Skip to content

Commit

Permalink
refactor(bb/Dockerfile): remove specific deps preparation step
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
ccfontes committed Dec 20, 2023
1 parent 15fb02e commit c0acc1f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
5 changes: 2 additions & 3 deletions template/bb/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,12 @@ RUN addgroup --system app && adduser --system --ingroup app app
USER app
WORKDIR $HOME

RUN bb prepare -Sdeps '{:deps {prismatic/plumbing {:mvn/version "0.6.0"}}}'

COPY --chown=app:app index.clj bb.edn lib merge_config.clj ./

RUN bb prepare

COPY --chown=app:app function/bb.edn* function/bb.edn
RUN bb merge-config --src "function/bb.edn" --out "ship.edn" && \
mkdir ~/.m2 && \
bb --config ship.edn prepare && \
bb --config ship.edn print-deps && \
cp -r .m2 .m2-ship
Expand Down
1 change: 1 addition & 0 deletions template/bb/bb.edn
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{:paths ["." "function"]
:deps {prismatic/plumbing {:mvn/version "0.6.0"}}
:tasks {run-function index/-main
test test.run-tests/-main
merge-config merge-config/-main}}

0 comments on commit c0acc1f

Please sign in to comment.