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: provide an internal tools module #2922

Closed
wants to merge 5 commits into from

Conversation

mmorel-35
Copy link
Contributor

What does this PR do?

Provides an internal tools module that isolates tools in a dedicated folder (.tools) at the root of the folder instead of using local GOBIN

Why is it important?

This helps sharing the same versions for tools between developers. It also allow to use dependencies managers to provide updates on those tools.

This is inspired by the work done on opentelemetry-collector

@mmorel-35 mmorel-35 requested a review from a team as a code owner December 17, 2024 07:15
Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit b29a3b3
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/6761f613f130820008feea2a
😎 Deploy Preview https://deploy-preview-2922--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.

@mmorel-35 mmorel-35 marked this pull request as draft December 17, 2024 12:50
@stevenh
Copy link
Collaborator

stevenh commented Dec 17, 2024

Is this impacted by the impending tools addition to golang as detailed here?

@mmorel-35
Copy link
Contributor Author

It's probably better to wait for this feature as the v2 of mockery it getting in the way of the script here.
Except if you know a way with Makefile already 😉

@mmorel-35 mmorel-35 closed this Dec 17, 2024
@mmorel-35 mmorel-35 deleted the tools branch December 17, 2024 22:35
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