Skip to content

Conversation

@mdelapenya
Copy link
Member

What does this PR do?

It adds the filter for the modified modules to the codeql workflow

Why is it important?

The workflow is analysing all the repository with every PR, when it's not needed, adding around 6 mins to the build.

With this change we expect to reduce that build time.

@mdelapenya mdelapenya requested a review from a team as a code owner April 14, 2025 10:27
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Apr 14, 2025
@mdelapenya mdelapenya self-assigned this Apr 14, 2025
@netlify
Copy link

netlify bot commented Apr 14, 2025

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit a814a54
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/67fce2f791ba7f0008c80711
😎 Deploy Preview https://deploy-preview-3103--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mdelapenya mdelapenya merged commit 314983a into testcontainers:main Apr 14, 2025
499 of 500 checks passed
@mdelapenya mdelapenya deleted the codeql-modules branch April 14, 2025 13:30
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Apr 15, 2025
* main:
  chore(deps): bump golang.org/x/crypto in /modules/aerospike (testcontainers#3105)
  chore(ci): run codeql on the modified modules (testcontainers#3103)
  docs: fix reference to container types in Run function (testcontainers#3102)
  chore(deps): bump github.com/golang-jwt/jwt/v5 in /modules/pulsar (testcontainers#3101)
  chore(pulsar): bump github.com/apache/pulsar-client-go from 0.10.0 to 0.14.0 (testcontainers#3100)
  chore(clickhouse): bump github.com/ClickHouse/clickhouse-go/v2 from 2.20.0 to 2.34.0 in /modules/clickhouse (testcontainers#3099)
  chore(deps): bump github/codeql-action from 3.28.13 to 3.28.15 (testcontainers#3097)
  chore(deps): bump golang.org/x/crypto from 0.31.0 to 0.37.0 (testcontainers#3098)
  feat(aerospike): add Aerospike module (testcontainers#3094)
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Apr 16, 2025
mdelapenya added a commit that referenced this pull request Apr 16, 2025
* Revert "chore(ci): reduce GH runners usage by calling codeql in the lint stage (#3108)"

This reverts commit a0ccdbb.

* Revert "chore(ci): run codeql on the modified modules (#3103)"

This reverts commit 314983a.

* chore: remove javascript from codeql action
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Changes that do not impact the existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant