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

feat: rm leiningen #164

Merged
merged 35 commits into from
Jul 22, 2024
Merged

feat: rm leiningen #164

merged 35 commits into from
Jul 22, 2024

Conversation

erdos
Copy link
Owner

@erdos erdos commented Jul 18, 2024

Tasks:

  • Remove usage of Leiningen build tool in favor of tools.deps
  • Introduce Kaocha test runner

TODO:

  • build single JAR file (with pom.xml)
  • build uberjar file (with manifest), ensure that standalone mode works.
  • run clojure unit tests
  • run clojure visual regression tests
  • run java junit tests
  • tests report results in XML format (no change here)
  • build javadoc pages
  • basic commands: clean, compile, jar, uberjar, etc.
  • check: version numbering works as expected
  • check: spec instrumentation works as expected
  • publishing to Clojars works without any changes
  • changes in documentation
  • ...

Copy link

codecov bot commented Jul 18, 2024

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 93.75%. Comparing base (43bd502) to head (496693a).

Files Patch % Lines
src/stencil/spec.clj 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #164      +/-   ##
==========================================
+ Coverage   93.40%   93.75%   +0.34%     
==========================================
  Files          31       31              
  Lines        2093     2097       +4     
  Branches      100       93       -7     
==========================================
+ Hits         1955     1966      +11     
  Misses         38       38              
+ Partials      100       93       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@erdos erdos marked this pull request as ready for review July 22, 2024 16:17
@erdos erdos merged commit 901c662 into master Jul 22, 2024
6 checks passed
@erdos erdos deleted the rm-lein branch July 22, 2024 16:19
erdos added a commit that referenced this pull request Jul 22, 2024
fix: tests in main flow
erdos added a commit that referenced this pull request Jul 22, 2024
fix: tests in main flow
erdos added a commit that referenced this pull request Jul 22, 2024
fix: tests in main flow
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