Skip to content

Fix some bugs with the actions required thingy#36

Merged
roryabraham merged 3 commits intomainfrom
d-param-required
May 7, 2025
Merged

Fix some bugs with the actions required thingy#36
roryabraham merged 3 commits intomainfrom
d-param-required

Conversation

@rafecolton
Copy link
Member

Details

Related Issues

Part of https://github.com/Expensify/Expensify/issues/484931

Manual Tests

Testing in https://github.com/Expensify/Salt/pull/14890

Linked PRs

N/A

@rafecolton rafecolton requested a review from roryabraham May 7, 2025 01:57
@rafecolton rafecolton self-assigned this May 7, 2025
@github-actions
Copy link

github-actions bot commented May 7, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@rafecolton
Copy link
Member Author

I have read the CLA Document and I hereby sign the CLA

@roryabraham roryabraham merged commit 531bce6 into main May 7, 2025
4 of 5 checks passed
@roryabraham roryabraham deleted the d-param-required branch May 7, 2025 02:02
@melvin-bot
Copy link

melvin-bot bot commented May 7, 2025

@roryabraham looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

@melvin-bot melvin-bot bot added the Emergency label May 7, 2025
@roryabraham
Copy link
Contributor

not an emergency - chicken and egg. The repository ruleset was failing on main (due to missing description). CLA action wants to write JSON to main, but it can't because the ruleset is failing on main.

Rafe tried to sign the CLA

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.

2 participants