Skip to content

chore: run make lint on new modules#2122

Merged
mdelapenya merged 3 commits intotestcontainers:mainfrom
mdelapenya:make-lint-modules
Jan 17, 2024
Merged

chore: run make lint on new modules#2122
mdelapenya merged 3 commits intotestcontainers:mainfrom
mdelapenya:make-lint-modules

Conversation

@mdelapenya
Copy link
Member

  • chore: run make lint for new modules
  • chore: add readability in the generate function

What does this PR do?

It automatically runs make lint on new modules created with the module generator tool.

Why is it important?

New module authors could contribute a module but forget to run the lint, ending up in failures in the CI.

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner January 17, 2024 11:53
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Jan 17, 2024
@mdelapenya mdelapenya self-assigned this Jan 17, 2024
@netlify
Copy link

netlify bot commented Jan 17, 2024

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit c114222
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/65a7bfc578b03a0007a1f20d
😎 Deploy Preview https://deploy-preview-2122--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 b6b4e8c into testcontainers:main Jan 17, 2024
mdelapenya added a commit that referenced this pull request Jan 17, 2024
….com/docker/compose/v2-2.24.0

* main:
  chore: run make lint on new modules (#2122)
  enable golangci-lint for pulsar (#2121)
  lint: enable testifylint (#2120)
  chore(deps): bump github.com/nats-io/nats.go in /modules/nats (#2094)
  chore(deps): bump golang.org/x/sys from 0.15.0 to 0.16.0 (#2104)
  Revert "chore(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (#2088)"
  chore(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (#2088)
  chore(deps): bump cloud.google.com/go/spanner from 1.54.0 to 1.55.0, google.golang.org/api from 0.154.0 to 0.156.0 in /modules/gcloud (#2115)
  chore(deps): bump github.com/aws/aws-sdk-go-v2/config from 1.25.10 to 1.26.3, github.com/aws/aws-sdk-go from 1.48.13 to 1.49.19 in /modules/localstack (#2114)
  chore(deps): bump github.com/docker/go-connections from 0.4.0 to 0.5.0 (#2113)
mdelapenya added a commit to jespino/testcontainers-go that referenced this pull request Jan 18, 2024
* main:
  enable golangci-lint for redis module (testcontainers#2126)
  Go install gotestsum and golangci-lint  (testcontainers#2127)
  improve OSSF score (testcontainers#2125)
  chore: run make lint on new modules (testcontainers#2122)
  enable golangci-lint for pulsar (testcontainers#2121)
  lint: enable testifylint (testcontainers#2120)
  chore(deps): bump github.com/nats-io/nats.go in /modules/nats (testcontainers#2094)
  chore(deps): bump golang.org/x/sys from 0.15.0 to 0.16.0 (testcontainers#2104)
  Revert "chore(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (testcontainers#2088)"
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Jan 24, 2024
* main: (25 commits)
  chore(deps): bump github.com/dvsekhvalnov/jose2go in /modules/pulsar (testcontainers#2136)
  fix: skip-host-cache option removed in latest MySQL 8.3.0 version (testcontainers#2130)
  chore: skip assertions for Docker Rootless (testcontainers#2135)
  pin Docker images version (testcontainers#2129)
  enable golangci-lint for examples (testcontainers#2128)
  chore(deps): bump github.com/neo4j/neo4j-go-driver/v5 in /modules/neo4j (testcontainers#2098)
  enable golangci-lint for redis module (testcontainers#2126)
  Go install gotestsum and golangci-lint  (testcontainers#2127)
  improve OSSF score (testcontainers#2125)
  chore: run make lint on new modules (testcontainers#2122)
  enable golangci-lint for pulsar (testcontainers#2121)
  lint: enable testifylint (testcontainers#2120)
  chore(deps): bump github.com/nats-io/nats.go in /modules/nats (testcontainers#2094)
  chore(deps): bump golang.org/x/sys from 0.15.0 to 0.16.0 (testcontainers#2104)
  Revert "chore(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (testcontainers#2088)"
  chore(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (testcontainers#2088)
  chore(deps): bump cloud.google.com/go/spanner from 1.54.0 to 1.55.0, google.golang.org/api from 0.154.0 to 0.156.0 in /modules/gcloud (testcontainers#2115)
  chore(deps): bump github.com/aws/aws-sdk-go-v2/config from 1.25.10 to 1.26.3, github.com/aws/aws-sdk-go from 1.48.13 to 1.49.19 in /modules/localstack (testcontainers#2114)
  chore(deps): bump github.com/docker/go-connections from 0.4.0 to 0.5.0 (testcontainers#2113)
  Adding mockserver module (testcontainers#2085)
  ...
@mdelapenya mdelapenya deleted the make-lint-modules branch January 29, 2024 13:07
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