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.

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

Enable setting configuration options for local testing #290

Merged
merged 12 commits into from Oct 24, 2022
Merged

Conversation

cnagadya
Copy link
Contributor

@cnagadya cnagadya commented Oct 14, 2022

Adds ability to set configuration options via dev-config.yml

PS: With modification of CONTRIBUTING.md to update instructions, there are some linter changes included in this PR

@cnagadya cnagadya changed the title Retrieve config file values for local testing Enable setting configuration options locally Oct 14, 2022
@cnagadya cnagadya changed the title Enable setting configuration options locally Enable setting configuration options for local testing Oct 14, 2022
@cnagadya cnagadya marked this pull request as ready for review October 14, 2022 12:49
@cnagadya cnagadya requested a review from a team as a code owner October 14, 2022 12:49
CONTRIBUTING.md Outdated Show resolved Hide resolved
scripts/scan_pr Show resolved Hide resolved
scripts/scan_pr Outdated Show resolved Hide resolved
@febuiles
Copy link
Contributor

Validation:

Screenshot 2022-10-21 at 14 05 05

With an external config file:

Screenshot 2022-10-21 at 14 09 07

@febuiles
Copy link
Contributor

@cnagadya please take a look and let me know what you think!

scripts/scan_pr Outdated Show resolved Hide resolved
scripts/scan_pr Outdated Show resolved Hide resolved
@cnagadya
Copy link
Contributor Author

Looks great. Just two small (script vs scripts) typos in scan_pr

febuiles and others added 2 commits October 24, 2022 16:53
Co-authored-by: cnagadya <cnagadya@github.com>
Co-authored-by: cnagadya <cnagadya@github.com>
@febuiles febuiles merged commit d4f6425 into main Oct 24, 2022
@cnagadya cnagadya deleted the cn/scan_pr branch October 24, 2022 15:18
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.

None yet

2 participants