-
Notifications
You must be signed in to change notification settings - Fork 17
thank you #268
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
base: main
Are you sure you want to change the base?
thank you #268
Conversation
Bumps the npm_and_yarn group with 1 update in the / directory: [follow-redirects](https://github.com/follow-redirects/follow-redirects). Updates `follow-redirects` from 1.15.2 to 1.15.9 - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.2...v1.15.9) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <[email protected]>
…nd_yarn-9f7fd066c2 build(deps-dev): bump follow-redirects from 1.15.2 to 1.15.9 in the npm_and_yarn group across 1 directory
Reviewer's Guide by SourceryThis pull request introduces a CodeQL workflow to enable static analysis for JavaScript/TypeScript and Python code. The workflow is configured to run on push and pull requests to the main branch, as well as on a weekly schedule. The results of the analysis are categorized by language. [FILTERED - Architecture diagrams are low quality] Architecture diagram for CodeQL integrationgraph LR
subgraph GitHub Actions
A[Push/Pull Request] --> B(CodeQL Workflow)
B --> C{Initialize CodeQL}
C --> D{Perform CodeQL Analysis}
D --> E[Security Events]
end
subgraph Repository
F[JavaScript/TypeScript Code]
G[Python Code]
end
E --> H((GitHub Security))
C --> F
C --> G
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Bumps the npm_and_yarn group with 1 update in the / directory: [@vscode/vsce](https://github.com/Microsoft/vsce). Updates `@vscode/vsce` from 2.18.0 to 3.3.1 - [Release notes](https://github.com/Microsoft/vsce/releases) - [Commits](microsoft/vscode-vsce@v2.18.0...v3.3.1) --- updated-dependencies: - dependency-name: "@vscode/vsce" dependency-type: direct:development dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps the npm_and_yarn group with 1 update in the / directory: [braces](https://github.com/micromatch/braces). Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](micromatch/braces@3.0.2...3.0.3) --- updated-dependencies: - dependency-name: braces dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps the npm_and_yarn group with 1 update in the / directory: [postcss](https://github.com/postcss/postcss). Updates `postcss` from 8.4.24 to 8.5.3 - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.24...8.5.3) --- updated-dependencies: - dependency-name: postcss dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @JohnDaWalka - I've reviewed your changes - here's some feedback:
Overall Comments:
- Consider using a single language entry in the matrix if the build mode is the same for both javascript-typescript and python.
Here's what I looked at during the review
- 🟡 General issues: 3 issues found
- 🟢 Security: all looks good
- 🟢 Review instructions: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Sourcery is free for open source - if you like our reviews please consider sharing them ✨
…nd_yarn-b3b4e61674
…d_yarn-c120d6abdd
…d_yarn-5134b82be1
Bumps the npm_and_yarn group with 1 update in the / directory: [esbuild](https://github.com/evanw/esbuild). Updates `esbuild` from 0.18.10 to 0.25.0 - [Release notes](https://github.com/evanw/esbuild/releases) - [Changelog](https://github.com/evanw/esbuild/blob/main/CHANGELOG-2023.md) - [Commits](evanw/esbuild@v0.18.10...v0.25.0) --- updated-dependencies: - dependency-name: esbuild dependency-type: direct:development dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <[email protected]>
…d_yarn-392d5e510b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good
Checklist
package.json
oryarn.lock
have changed, then test the VSIX built byyarn run vsce package
works from a direct installSummary by Sourcery
CI: