-
Notifications
You must be signed in to change notification settings - Fork 1k
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
docs: fill all command line flags not in the document, and more. #2568
Conversation
@@ -114,6 +114,10 @@ the Atlantis user needs to be part of that list in order for it to consider | |||
a pull request mergeable. | |||
::: | |||
|
|||
::: warning | |||
If you set `atlantis/apply` to the mergeable requirement, use `--gh-allow-mergeable-bypass-apply` flag. This flag allows the mergeable check before `apply` to skip checking the status of `atlantis/apply`. |
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.
Can we also document the ATLANTIS_GH_MERGEABLE_BYPASS_APPLY=true
env var too ?
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.
Looks like the flag is documented here https://www.runatlantis.io/docs/server-configuration.html#gh-allow-mergeable-bypass-apply but it doesnt show the env var.
It also shows a bullet where there shouldn't be a bullet.
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.
@nitrocode Thanks for your comment!
I have fixed a wrong bullet and documented the env var to apply-requirements.md
. On the other hand, I have not appended the env var to server-configuration.md
because there is no mention of env vars in the other flags (Excluding a few. Lack uniformity...).
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.
Ah I had a small typo #2583
@chroju Thanks for the contribution |
…atlantis#2568) * docs: All command line flags are filled in the doc * docs: about --gh-allow-mergeable-bypass-apply flag * docs: replace Github with GitHub * docs: remove an incorrect bullet Co-authored-by: chroju <chroju@users.noreply.github.com>
Add documents.
server/server.go
, related:--hide-prev-plan-comments
flag fails if using Github App name with uppercased or space chars #1161 .atlantis/apply
status check work withmergeable
using--gh-allow-mergeable-bypass-apply
#2436 .