Skip to content

Init CLI flag documentation improvement#11721

Open
emilypi wants to merge 9 commits intohaskell:masterfrom
emilypi:emily/init-docs-improvements
Open

Init CLI flag documentation improvement#11721
emilypi wants to merge 9 commits intohaskell:masterfrom
emilypi:emily/init-docs-improvements

Conversation

@emilypi
Copy link
Copy Markdown
Member

@emilypi emilypi commented Apr 9, 2026

See @Mikolaj's comment. The existing docs are there for anyone who issues --help, but it's non-obvious to Haskell developers that they may need help, so this creates a natural conflict.

Copy link
Copy Markdown
Collaborator

@ulysses4ever ulysses4ever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think shoving more stuff in the --help isn't very helpful (although isn't harmful either, so I approve). My impression from that issue thread was that the agreement was that it's the interactive mode who should advertislze the other mode in the very beginning.

cc @AndreasPK

Copy link
Copy Markdown
Collaborator

@geekosaur geekosaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm neutral on this, FWIW, and agree that we should consider splitting functionality at some point as something more useful than just adjusting help text.

I will note that some programs (notably most of Larry Wall's: original patch, rn, etc.) output their manpages for --help. (But not Perl, because it's a full language and there's like 30 of them at this point and were at least 10 at the original public release.) That's probably overkill, though.

@emilypi
Copy link
Copy Markdown
Member Author

emilypi commented Apr 9, 2026

At the very least, the documentation was stale, so this brings it up to par before we branch off or do whatever.

@ulysses4ever
Copy link
Copy Markdown
Collaborator

Emily, I should have said first that I appreciate your effort before jumping into the negative part. I apologize!

I didn't notice any fixes per se: I thought this patch was purely additive. But I see it now: the part about supplying the root of the project was removed. Thanks!

But since you mentioned documentation, I think these days (and especially due to the AI storm) the online docs are much more important than --help. In general, when updating one, I'd expect an update to the other as well... I have checked https://cabal.readthedocs.io/en/stable/cabal-commands.html#cabal-init now and, sure enough, it looks outdated as well because it appears to be implying the noninteractive default.

@emilypi
Copy link
Copy Markdown
Member Author

emilypi commented Apr 9, 2026

I actually don't know how to modify the readthedocs site fwiw - point me at the repo and I'm sure I'll survive. I can add it to the PR. I don't mind criticism btw - we're in subjective territory here

@ulysses4ever
Copy link
Copy Markdown
Collaborator

@emilypi
Copy link
Copy Markdown
Member Author

emilypi commented Apr 9, 2026

@ulysses4ever thanks! updated.

@ulysses4ever
Copy link
Copy Markdown
Collaborator

Great! The formatting and whitespace jobs are failing which is the last thing that needs fixing before the merge bot can take it.

@emilypi
Copy link
Copy Markdown
Member Author

emilypi commented Apr 24, 2026

@ulysses4ever should be fixed now. Nice tool btw

@ulysses4ever
Copy link
Copy Markdown
Collaborator

formatting seems unhappy again, I'm sorry! We use fourmolu-0.12.0.0. An older version always bites people...

also, you can apply a merge label when you're happy with it. The squash label is appropriate here perhaps.

@emilypi emilypi added the merge me Tell Mergify Bot to merge label Apr 24, 2026
@emilypi
Copy link
Copy Markdown
Member Author

emilypi commented Apr 24, 2026

I'm a fan of the new checks!

@mergify mergify Bot added ready and waiting Mergify is waiting out the cooldown period merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days queued labels Apr 24, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 26, 2026

Merge Queue Status

This pull request spent 1 hour 40 minutes 55 seconds in the queue, including 1 hour 30 minutes 40 seconds running CI.

Waiting for any of:
  • check-neutral = Validate post job
  • check-skipped = Validate post job
  • check-success = Validate post job
All conditions
  • any of [🛡 GitHub branch protection]:
    • check-neutral = Validate post job
    • check-skipped = Validate post job
    • check-success = Validate post job
  • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • any of [🛡 GitHub branch protection]:
    • check-success = Doctest Cabal
    • check-neutral = Doctest Cabal
    • check-skipped = Doctest Cabal
  • any of [🛡 GitHub branch protection]:
    • check-success = Meta checks
    • check-neutral = Meta checks
    • check-skipped = Meta checks
  • any of [🛡 GitHub branch protection]:
    • check-success = docs/readthedocs.org:cabal
    • check-neutral = docs/readthedocs.org:cabal
    • check-skipped = docs/readthedocs.org:cabal
  • any of [🛡 GitHub branch protection]:
    • check-success = fourmolu
    • check-neutral = fourmolu
    • check-skipped = fourmolu
  • any of [🛡 GitHub branch protection]:
    • check-success = hlint
    • check-neutral = hlint
    • check-skipped = hlint
  • any of [🛡 GitHub branch protection]:
    • check-success = Bootstrap post job
    • check-neutral = Bootstrap post job
    • check-skipped = Bootstrap post job
  • any of [🛡 GitHub branch protection]:
    • check-success = whitespace
    • check-neutral = whitespace
    • check-skipped = whitespace
  • any of [🛡 GitHub branch protection]:
    • check-success = Check sdist post job
    • check-neutral = Check sdist post job
    • check-skipped = Check sdist post job
  • any of [🛡 GitHub branch protection]:
    • check-success = Changelogs
    • check-neutral = Changelogs
    • check-skipped = Changelogs

Reason

The merge conditions cannot be satisfied due to failing checks

Failing checks:

Hint

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio queue comment.

mergify Bot added a commit that referenced this pull request Apr 26, 2026
@mergify mergify Bot added dequeued waiting too long automatically set by Mergify to trigger a warning and removed queued labels Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dequeued merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days merge me Tell Mergify Bot to merge ready and waiting Mergify is waiting out the cooldown period waiting too long automatically set by Mergify to trigger a warning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants