Skip to content

Conversation

@tdschwarz
Copy link

@tdschwarz tdschwarz commented Dec 16, 2025

This change enables auto-generating marker files during releases, allowing 'go mod vendor' to work without manual intervention.

Implementation:

  • Update .gitignore to ignore tools.go
  • Modify go-cd.yml to generate doc.go for each platform during release
  • Modify go-cd.yml to generate tools.go with all platform imports

Issue link

This Pull Request is linked to issue (URL): #4721

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one issue.
  • Commit message has a detailed description of what changed and why.
  • [N/A] Tests are added or updated.
  • CHANGELOG.md and documentation files are updated.
  • Linters have been run (make *-lint targets) and Prettier has been run (make prettier-fix).
  • Destination branch is correct - main or release
  • Create merge commit if merging release branch into main, squash otherwise.

@tdschwarz tdschwarz requested a review from a team as a code owner December 16, 2025 00:28
@tdschwarz tdschwarz force-pushed the feat/go-vendor-support branch from 7fd1ffb to 95581dc Compare December 16, 2025 16:23
@xShinnRyuu
Copy link
Collaborator

@tdschwarz Please resolve CICD lint errors

@tdschwarz
Copy link
Author

@tdschwarz Please resolve CICD lint errors

@xShinnRyuu done!

@tdschwarz tdschwarz force-pushed the feat/go-vendor-support branch 2 times, most recently from cf64bb2 to 0edad1e Compare December 18, 2025 05:12
This change enables auto-generating marker files during releases,
allowing 'go mod vendor' to work without manual intervention.

Implementation:
- Update .gitignore to ignore tools.go
- Modify go-cd.yml to generate doc.go for each platform during release
- Modify go-cd.yml to generate tools.go with all platform imports

Signed-off-by: tdschwarz <[email protected]>
@tdschwarz tdschwarz force-pushed the feat/go-vendor-support branch from 84cfb4c to ca9ae85 Compare December 29, 2025 16:00
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