Skip to content

Add support for environment > deployment: false#639

Closed
springmeyer wants to merge 2 commits intorhysd:mainfrom
springmeyer:springmeyer/deployment-false
Closed

Add support for environment > deployment: false#639
springmeyer wants to merge 2 commits intorhysd:mainfrom
springmeyer:springmeyer/deployment-false

Conversation

@springmeyer
Copy link
Copy Markdown
Contributor

@springmeyer
Copy link
Copy Markdown
Contributor Author

@rhysd just also added timezone support as well to this PR. This was also added in https://github.blog/changelog/2026-03-19-github-actions-late-march-2026-updates/#github-actions-now-allows-developers-to-use-environments-without-auto-deployment and I'm seeing false-positives when testing it (before this PR).

@martincostello
Copy link
Copy Markdown
Contributor

@springmeyer #641 also implements timezone support 😉

@rhysd
Copy link
Copy Markdown
Owner

rhysd commented Mar 30, 2026

I'm sorry for the delay. I couldn't take enough time to review this.

@gplessis
Copy link
Copy Markdown

@springmeyer #641 has been merged for the timeline support, but environment.deployment still isn't supported. Should we re-open this PR, with no timezone support ?

@rhysd
Copy link
Copy Markdown
Owner

rhysd commented Mar 30, 2026

Yes, I notice it. I'm cherry-picking implementation in this branch now. Thanks anyway.

@rhysd
Copy link
Copy Markdown
Owner

rhysd commented Mar 30, 2026

I cherry-picked the implementation at 0ef3e18.

@springmeyer
Copy link
Copy Markdown
Contributor Author

thanks @rhysd

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.

4 participants