Skip to content

Fix recipe doc alignment in just --list#1816

Merged
casey merged 4 commits intocasey:masterfrom
crdx:alignment
Jan 9, 2024
Merged

Fix recipe doc alignment in just --list#1816
casey merged 4 commits intocasey:masterfrom
crdx:alignment

Conversation

@crdx
Copy link
Contributor

@crdx crdx commented Jan 3, 2024

recipe.private doesn't take into account recipes with the [private] attribute, so private recipes affect the alignment calculation of the the recipe docs. This causes them to align too far to the right.

Use recipe.public() instead which appears to be the canonical way of differentiating between public and private.

`recipe.private` doesn't take into account recipes with the `[private]`
attribute, so private recipes affect the alignment calculation of the
the recipe docs. This causes them to align too far to the right.

Use `recipe.public()` instead which appears to be the canonical way of
differentiating between public and private.
@casey
Copy link
Owner

casey commented Jan 3, 2024

Nice, good catch! Can you add a test for this?

@crdx
Copy link
Contributor Author

crdx commented Jan 4, 2024

Yep. Should it go in tests/modules.rs or a new file tests/subcommand.rs?

@crdx
Copy link
Contributor Author

crdx commented Jan 4, 2024

I went ahead and added it to tests/modules.rs as that felt like the right place. Verified it fails before the change and passes after.

@casey casey enabled auto-merge (squash) January 9, 2024 08:06
@casey
Copy link
Owner

casey commented Jan 9, 2024

Whoops, sorry for forgetting about this. Looks good to me!

@casey casey merged commit 541e781 into casey:master Jan 9, 2024
@crdx crdx deleted the alignment branch January 10, 2024 22:13
casey pushed a commit that referenced this pull request Jan 12, 2024
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.

2 participants