Skip to content

(CAT-2624) Adjusting token logic#159

Merged
gavindidrichsen merged 3 commits intomainfrom
CAT-2624-adjusting_token_logic
Apr 30, 2026
Merged

(CAT-2624) Adjusting token logic#159
gavindidrichsen merged 3 commits intomainfrom
CAT-2624-adjusting_token_logic

Conversation

@LukasAud
Copy link
Copy Markdown
Contributor

@LukasAud LukasAud commented Apr 29, 2026

Make forge auth and Twingate steps conditional for fork PR support

Move token setup out of top-level env into conditional shell steps so that empty strings are not passed into the Gemfile, where they can be interpreted as 'truthy'. Guard Twingate steps so repos without the secret skip them gracefully.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.

Make forge auth and Twingate steps conditional for fork PR support

Move token setup out of top-level env into conditional shell steps, and
guard Twingate steps so repos without the secret skip them gracefully.
@LukasAud LukasAud requested review from a team as code owners April 29, 2026 08:34
@LukasAud LukasAud added the bug Something isn't working label Apr 29, 2026
@gavindidrichsen gavindidrichsen merged commit 444a423 into main Apr 30, 2026
2 checks passed
@gavindidrichsen gavindidrichsen deleted the CAT-2624-adjusting_token_logic branch April 30, 2026 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants