Skip to content

fix: crate langugages config file for local mode#136

Merged
andrzej-janczak merged 3 commits intomainfrom
fix/create-languages-config-for-local-mode
May 29, 2025
Merged

fix: crate langugages config file for local mode#136
andrzej-janczak merged 3 commits intomainfrom
fix/create-languages-config-for-local-mode

Conversation

@andrzej-janczak
Copy link
Copy Markdown
Contributor

@andrzej-janczak andrzej-janczak commented May 29, 2025

No description provided.

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented May 29, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-0.22% 0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (def7b8e) 4541 1430 31.49%
Head commit (533ca6b) 4573 (+32) 1430 (+0) 31.27% (-0.22%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#136) 32 0 0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Comment thread cmd/init.go Outdated
Comment on lines +81 to +85
content := `tools:
- name: semgrep
languages: [CPP, Go, Java, JavaScript, JSON, Python, Ruby]
extensions: [.cc, .cpcac, .cpp, .cxx, .gemspec, .go, .hpp, .ino, .java, .jbuilder, .js, .jsm, .json, .jssssx, .jsx, .mjs, .opal, .podspec, .py, .pyx, .rake, .rb, .vue]
- name: trivy
Copy link
Copy Markdown
Contributor

@zhamborova zhamborova May 29, 2025

Choose a reason for hiding this comment

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

I think we are missing cpp, dart analyzer mappings

Comment thread cmd/init.go
},
}

func createLanguagesConfigFileLocal(toolsConfigDir string) error {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

missing dart?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes, dart

Comment thread cmd/init.go Outdated
Comment on lines +97 to +99
- name: eslint
languages: [JavaScript]
extensions: [.js, .jsm, .jsx, .mjs, .vue]`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

typescript?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added

@andrzej-janczak andrzej-janczak enabled auto-merge (squash) May 29, 2025 11:54
@andrzej-janczak andrzej-janczak merged commit aa560d8 into main May 29, 2025
10 checks passed
@alerizzo alerizzo deleted the fix/create-languages-config-for-local-mode branch June 3, 2025 09:45
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.

3 participants