-
Notifications
You must be signed in to change notification settings - Fork 18
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
Misc Fixes #2125
base: main
Are you sure you want to change the base?
Misc Fixes #2125
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: Maintain the same style in:
/dev/blaze/dev.clj
, and in/profiling/blaze/profiling.edn
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the future, find a way to scan all files with parinfer.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2125 +/- ##
=======================================
Coverage 94.99% 94.99%
=======================================
Files 330 330
Lines 20158 20158
Branches 482 482
=======================================
Hits 19149 19149
Misses 527 527
Partials 482 482 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: review REPL aliases.
- assess whether it is possible and desirable to add the
test
path to the global REPL, so we don't need per-module explicit REPL setup anymore.
@@ -4,7 +4,7 @@ $(MODULES): | |||
$(MAKE) -C $@ $(MAKECMDGOALS) | |||
|
|||
fmt-root: | |||
cljfmt check dev resources src test deps.edn tests.edn | |||
cljfmt check dev modules profiling resources src test deps.edn tests.edn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need the check the whole modules dir here, because that is done at module level. profiling
is ok.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And I miss the fixed profiling namespace.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And I miss the fixed profiling namespace.
See TODO here:
#2125 (comment)
ca41c01
to
fdefc4b
Compare
modules
andprofiling
foldersdocs/implementation/fhir-data-model.md