Skip to content

Conversation

sepetrov
Copy link
Contributor

@sepetrov sepetrov commented Sep 30, 2025

Changes the comment if linting issues are found so that:

  • golangci-lint fixes the issues found
  • uses the main branch as a reference so that only changes are inspected
  • includes GitHub PAT and initialises git so that private packages can be downloaded in the Docker container on demand

@sepetrov
Copy link
Contributor Author

Testing

The workflow is tested in https://github.com/Typeform/integrations-monorepo/actions/runs/18123503391/job/51573683076

Screenshot 2025-09-30 at 11 33 51 Screenshot 2025-09-30 at 11 47 12

Note

I had to change the working directory in the command to the directory of the Go module, which I wanted to fix the linting issues for.

@sepetrov sepetrov marked this pull request as ready for review September 30, 2025 08:54
@sepetrov sepetrov requested a review from a team as a code owner September 30, 2025 08:54
@sepetrov sepetrov changed the title chore(go-lint): comment fix command chore(go-lint): [NOJIRA-1234] comment fix command Sep 30, 2025
Copy link

gitstream-cm bot commented Sep 30, 2025

This PR is missing a Jira ticket reference in the title or description.
Please add a Jira ticket reference to the title or description of this PR.

Copy link

gitstream-cm bot commented Sep 30, 2025

🥷 Code experts: pmlanger

pmlanger has most 👩‍💻 activity in the files.
pmlanger has most 🧠 knowledge in the files.

See details

.github/workflows/go-lint-workflow.yaml

Activity based on git-commit:

pmlanger
SEP 33 additions & 13 deletions
AUG 1 additions & 1 deletions
JUL
JUN
MAY
APR

Knowledge based on git-blame:
pmlanger: 26%

✨ Comment /gs review for LinearB AI review. Learn how to automate it here.

@sepetrov sepetrov merged commit b80343e into main Sep 30, 2025
10 checks passed
@sepetrov sepetrov deleted the fix-lint-comment branch September 30, 2025 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants