Skip to content

feat: support creating and testing built-in checkers#23

Merged
sanket-deepsource merged 9 commits intomasterfrom
eng-1963
Feb 10, 2025
Merged

feat: support creating and testing built-in checkers#23
sanket-deepsource merged 9 commits intomasterfrom
eng-1963

Conversation

@sourya-deepsource
Copy link
Contributor

@sourya-deepsource sourya-deepsource commented Feb 7, 2025

This pull request adds the test workflow for built-in checkers, improving the handling of built-in checkers, and refining the analysis and CLI functionality. The most important changes are summarized below:

Build and Test Workflow:

  • Added a new GitHub Actions workflow to automate the tests for builtin checkers.

Checker and Rule Management:

  • Introduced a new LoadBuiltinCheckers function to load built-in checkers from embedded files.
  • Replaced the ReadCustomRules function with LoadYamlRules to support loading rules from both custom and built-in directories.

Language Support:

  • Expanded the LanguageFromFilePath function to support additional file extensions and added a new GetExtFromLanguage function to retrieve file extensions based on language.

CLI Enhancements:

  • Updated the CLI to allow specifying a directory for running tests and improved error messages for the --checkers flag.

Test Runner Improvements:

  • Enhanced the test runner to handle missing test cases gracefully and provided more informative output during test execution.

@vercel
Copy link

vercel bot commented Feb 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
globstar ⬜️ Ignored (Inspect) Visit Preview Feb 9, 2025 10:39am

Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
@sourya-deepsource sourya-deepsource changed the title chore: run checkers inside checkers/ dir feat: support creating and testing built-in checkers Feb 9, 2025
Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
@sanket-deepsource sanket-deepsource merged commit 8f2586f into master Feb 10, 2025
3 checks passed
@sanket-deepsource sanket-deepsource deleted the eng-1963 branch February 10, 2025 21:37
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