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

Fix recipe doc alignment in just --list #1816

Merged
merged 4 commits into from
Jan 9, 2024
Merged

Fix recipe doc alignment in just --list #1816

merged 4 commits into from
Jan 9, 2024

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
5 checks passed
@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