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

NEW: @W-15652762@: Add configuration parsing and validation abilities #13

Merged
merged 3 commits into from
May 16, 2024

Conversation

stephen-carter-at-sf
Copy link
Collaborator

@stephen-carter-at-sf stephen-carter-at-sf commented May 15, 2024

Adding in configuration parsing and validation which will allow clients to now pass in configuration data according the schema:

log_folder

  • Specify a location to write log files to. If not specified, then your machine’s default temporary directory will be used.

rules.<engine_name>.<rule_name>.severity

  • Specify the severity 1 (Critical), 2 (High), 3 (Moderate), 4 (Low), or 5 (Info) level as an integer that you want to set for the specified rule.

rules.<engine_name>.<rule_name>.tags

  • Specify a list of tags to set for the specified rule.

engines.<engine_name>.<engine_specific_property>

  • For the specified engine, you can specify any engine specific properties and values as needed.

Clients may pass in a yaml or json file, a yaml or json string, or an object.

@stephen-carter-at-sf stephen-carter-at-sf force-pushed the d/W-15652762 branch 3 times, most recently from 46d32b9 to a9ea81b Compare May 15, 2024 20:51
@stephen-carter-at-sf stephen-carter-at-sf changed the title NEW: @W-15652762@: Add configuration parsing abilities NEW: @W-15652762@: Add configuration parsing and validation abilities May 15, 2024
packages/code-analyzer-core/src/config.ts Outdated Show resolved Hide resolved
packages/code-analyzer-core/src/config.ts Outdated Show resolved Hide resolved
packages/code-analyzer-core/src/config.ts Outdated Show resolved Hide resolved
packages/code-analyzer-core/src/config.ts Show resolved Hide resolved
.github/workflows/verify-pr.yml Outdated Show resolved Hide resolved
.github/workflows/verify-pr.yml Outdated Show resolved Hide resolved
@stephen-carter-at-sf stephen-carter-at-sf merged commit cc0ad68 into dev May 16, 2024
9 checks passed
@stephen-carter-at-sf stephen-carter-at-sf deleted the d/W-15652762 branch May 16, 2024 17:46
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