-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Automate go.mod replace
statements
#7055
Comments
@Aneurysm9 you own the |
There is a separate tool used by the Go API/SDK that maintains replace directives everywhere, though it currently creates replaces for all modules in all modules, regardless whether they're actually used. So, yes, it's possible and there's a tool that can help, but it could use some improvement. |
@Aneurysm9 do you need additional help on making these improvements? |
Probably. I know @bryan-aguilar had expressed an interest in learning more about Go and this might be a good first project. |
|
"is not", or "is now"? |
Oops! Fixed! |
Can this issue remain open until crosslink is for the entire repo? Ideally this would be run in the CI as well to notify us when someone has missed replaced lines |
Definitely, sounds good to me! |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping |
…her string parts (open-telemetry#7055) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
This issue has been closed as inactive because it has been stale for 120 days with no activity. |
Currently the replace statements for dependencies local to this repository is done manually, it would be great to automate this, as it has caused pain in the past.
This was originally suggested in the discussion below:
@codeboten not sure if the replace statements were manually added, but would be great if the tool that updates the version for these local deps also add the replacements 🗡️
Originally posted by @bogdandrutu in #7021 (review)
The text was updated successfully, but these errors were encountered: