Skip to content

Conversation

lbarberi1927
Copy link
Collaborator

Resolves Issue 1757

@lbarberi1927 lbarberi1927 requested a review from wp99cp October 7, 2025 14:26
@lbarberi1927
Copy link
Collaborator Author

@wp99cp so far I just commented out the lines that check whether the action namespaces begin with rslethz/ and tested with an action from another namespace which seems to work... open for discussion if we want something more sophisticated, if I failed to consider something or if I should perform more tests

Copy link

github-actions bot commented Oct 7, 2025

Jest Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit fa31d10. ± Comparison against base commit 6705f0a.

♻️ This comment has been updated with latest results.

@wp99cp
Copy link
Member

wp99cp commented Oct 10, 2025

It would be nice to define an optional environment variable which can be used to define a regex. This regex should then be used to verify allowed actions. If the env is not set, all repositories should be allowed. Also, add this to the documentation.

Copy link
Member

@wp99cp wp99cp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as discussed on Friday

@lbarberi1927 lbarberi1927 marked this pull request as ready for review October 13, 2025 13:39
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