Skip to content
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

chore: run make lint on new modules #2122

Merged
merged 3 commits into from
Jan 17, 2024

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
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
120 checks passed
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