Skip to content

ci: add CodeQL#263

Closed
naorpeled wants to merge 3 commits intoalchemy-run:mainfrom
naorpeled:ci/add-codeql
Closed

ci: add CodeQL#263
naorpeled wants to merge 3 commits intoalchemy-run:mainfrom
naorpeled:ci/add-codeql

Conversation

@naorpeled
Copy link
Contributor

@naorpeled naorpeled commented Jun 2, 2025

This pull request introduces a CodeQL integration for the repository to enhance security and quality analysis. The main changes include the addition of a CodeQL configuration file and a GitHub Actions workflow to automate the analysis process.

CodeQL Integration:

  • .github/codeql/codeql-config.yml: Added a configuration file specifying paths to include/exclude in analysis, query packs, and custom query filters for CodeQL scans.

Automation via GitHub Actions:

  • .github/workflows/codeql-analysis.yml: Added a workflow to run CodeQL analysis on push and pull_request events for the main branch, as well as on a weekly schedule. Includes steps for repository checkout, dependency installation, and CodeQL analysis execution.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jun 2, 2025

Open in StackBlitz

npm i https://pkg.pr.new/sam-goodwin/alchemy@263

commit: 8424c20

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Comment on lines +61 to +66
# Install dependencies for examples that need separate installation
if [ -d "examples/cloudflare-tanstack-start" ]; then
cd examples/cloudflare-tanstack-start
bun install --frozen-lockfile
cd ../..
fi
Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure if this is necessary @sam-goodwin wdyt?

Copy link
Collaborator

Choose a reason for hiding this comment

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

tanstack start has a weird problem where it wasn't working in a monorepo. You could try adding it back to workspaces in package.json and see if that's still a problerm

Copy link
Contributor Author

@naorpeled naorpeled Jun 2, 2025

Choose a reason for hiding this comment

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

It seems to work without installing it individually, pushed the change 🙏

@naorpeled naorpeled marked this pull request as ready for review June 2, 2025 19:35
@sam-goodwin
Copy link
Collaborator

Closing this. Didn't have bandwidth to consider this as part of our workflow. Happy to re-visit.

@sam-goodwin sam-goodwin closed this Aug 7, 2025
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