Skip to content

fix: Add env vars for PR approval and mergeable status#5919

Merged
jamengual merged 2 commits intorunatlantis:mainfrom
filipenf:add-approved-mergeable-env-vars
Oct 31, 2025
Merged

fix: Add env vars for PR approval and mergeable status#5919
jamengual merged 2 commits intorunatlantis:mainfrom
filipenf:add-approved-mergeable-env-vars

Conversation

@filipenf
Copy link
Contributor

what

Adds 2 env vars ATLANTIS_PR_APPROVED and ATLANTIS_PR_MERGEABLE. This makes it easier for the shell command to decide if the command can proceed or not depending on the PR status

why

Makes it easier to enable custom workflows that depend on approval / mergeable status (see #5779)

In scenarios where there's custom logic needed on top of the built-in apply_requirements, this gives extra context to the shell command being executed

tests

references

#5779

@filipenf filipenf force-pushed the add-approved-mergeable-env-vars branch from b28c1ee to faabb60 Compare October 30, 2025 15:20
Adds 2 env vars ATLANTIS_PR_APPROVED and ATLANTIS_PR_MERGEABLE. This
makes it easier for the shell command to decide if the command can
proceed or not depending on the PR status

Signed-off-by: Filipe Felisbino <filipe@felisbino.dev>
@filipenf filipenf force-pushed the add-approved-mergeable-env-vars branch from faabb60 to 5f522bd Compare October 30, 2025 15:24
@filipenf filipenf marked this pull request as ready for review October 30, 2025 17:59
@dosubot dosubot bot added feature New functionality/enhancement go Pull requests that update Go code labels Oct 30, 2025
@github-actions github-actions bot removed the feature New functionality/enhancement label Oct 30, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 30, 2025
@jamengual jamengual merged commit a7b35a0 into runatlantis:main Oct 31, 2025
41 checks passed
@jamengual
Copy link
Contributor

thanks @filipenf for the contribution

dimisjim pushed a commit to dimisjim/atlantis that referenced this pull request Nov 2, 2025
)

Signed-off-by: Filipe Felisbino <filipe@felisbino.dev>
Co-authored-by: PePe Amengual <2208324+jamengual@users.noreply.github.com>
Signed-off-by: dimisjim <dimitris.moraitidis@gmail.com>
aidansteele pushed a commit to aidansteele/atlantis that referenced this pull request Mar 12, 2026
)

Signed-off-by: Filipe Felisbino <filipe@felisbino.dev>
Co-authored-by: PePe Amengual <2208324+jamengual@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update Go code lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants