Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow user to pass multiple exclude dirs #41

Closed
JacobDomagala opened this issue Feb 13, 2022 · 0 comments · Fixed by #51
Closed

Allow user to pass multiple exclude dirs #41

JacobDomagala opened this issue Feb 13, 2022 · 0 comments · Fixed by #51
Assignees
@JacobDomagala
Labels
enhancement New feature or request

Comments

@JacobDomagala
Copy link
Owner

No description provided.

@JacobDomagala JacobDomagala added the enhancement New feature or request label Feb 13, 2022
@JacobDomagala JacobDomagala self-assigned this Feb 13, 2022
@JacobDomagala JacobDomagala changed the title Allow user to pass multiple exlude dirs Allow user to pass multiple exclude dirs Feb 13, 2022
skrobinson added a commit to skrobinson/StaticAnalysis that referenced this issue Feb 17, 2022
@skrobinson
Replace Bash entry point with Python version
536332b 
This is a translation to Python with three semantic changes: implement
multiple exclude directories (issue JacobDomagala#41), disable pull_request event,
and directly use get_files_to_check.py logic rather than exec.

Multiple exclude directories are handled because it's easy to do in Python.
pull_request events are temporarily disabled as I am more interested in
the solving issues with pull_request_target event.

Signed-off-by: Sean Robinson <[email protected]>
JacobDomagala added a commit that referenced this issue Nov 16, 2022
@JacobDomagala
[#41]: Update get_files_to_check so it can be tested
8dbe54a
JacobDomagala added a commit that referenced this issue Nov 16, 2022
@JacobDomagala
[#41]: Create list from space separated string
215ea60
JacobDomagala added a commit that referenced this issue Nov 16, 2022
@JacobDomagala
[#41]: Fix wording in action.yml
08252c7
JacobDomagala added a commit that referenced this issue Nov 16, 2022
@JacobDomagala
[#41]: Add unittest for get_files_to_check.py
931fdbc
JacobDomagala added a commit that referenced this issue Nov 16, 2022
@JacobDomagala
[#41]: Run black on files
8a9f9ef
@JacobDomagala JacobDomagala mentioned this issue Nov 16, 2022
Merged
@JacobDomagala JacobDomagala linked a pull request Nov 16, 2022 that will close this issue
41: Allow user to pass multiple exclude dirs #51
Merged
JacobDomagala added a commit that referenced this issue Nov 16, 2022
@JacobDomagala
[#41]: Fix linter
f4fc449
JacobDomagala added a commit that referenced this issue Nov 16, 2022
@JacobDomagala
[#41]: Fix linter
98d8b3f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JacobDomagala JacobDomagala

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

41: Allow user to pass multiple exclude dirs JacobDomagala/StaticAnalysis
1 participant
@JacobDomagala