feat: add docs for justfile - #1648
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## justfile-fix #1648 +/- ##
=============================================
Coverage 96.34% 96.34%
=============================================
Files 46 46
Lines 3666 3666
=============================================
Hits 3532 3532
Misses 134 134 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
tpoliaw
left a comment
There was a problem hiding this comment.
The docs are good, thanks for adding them, mainly insure what should be here vs in the upstream projects.
|
Was there a reason for preferring the [doc('comment here')]
recipe:
...vs # comment here
recipe:
... |
Append `|| true` so the justfile can still be parsed/used on systems without a container runtime like docker or podman, instead of erroring out at command substitution time.
3935ec4 to
a4e36d1
Compare
Yes I personally like to make it explicit it is for documentation. Can use the docstring if you prefer it. When I was going through the docs There where a lot of, if you do it this way the docstring will come, if you do it this way it wont and doc approach was the one which had the least cognitive load in my mind |
tpoliaw
left a comment
There was a problem hiding this comment.
No strong preference either way
Shows docs. I have example a vscode plugin which has made some spacing changes.
I have also added just to completion in dev-container for ease of use