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.

Sign up for GitHub

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

Jump to bottom

Fix test workflow #24

Merged
merged 2 commits into from
Jan 10, 2024
Merged

Fix test workflow #24

merged 2 commits into from
Jan 10, 2024

Conversation

julianolf
Copy link
Contributor

@julianolf julianolf commented Jan 10, 2024 •
edited
Loading

Fix test workflow by:

  • Removing the deprecated tool golint.
  • Removing unnecessary tool gocov.
  • Removing Codecov integration step.
  • Updating setup-go and checkout actions.
  • Updating test command removing the flag that saves the coverage report to a file.

Fixes #23
Fixes #26

@julianolf julianolf added the enhancement New feature or request label Jan 10, 2024
@julianolf
Fix issues with other steps on test workflows.
c85543b 
- Update setup-go and checkout actions.
- Remove unnecessary gocov package installation.
- Update test command removing the flag that saves the coverage report
  to a file.
- Remove Codecov integration step.
@julianolf julianolf requested a review from a team January 10, 2024 19:02
@julianolf julianolf self-assigned this Jan 10, 2024
@julianolf julianolf merged commit 7cb7d01 into noverde:master Jan 10, 2024
2 checks passed
@julianolf julianolf deleted the fix/test-workflow branch January 10, 2024 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leonardoc-lima leonardoc-lima leonardoc-lima approved these changes

@Luis-Henrique Luis-Henrique Luis-Henrique approved these changes

@gabriel-f-santos gabriel-f-santos gabriel-f-santos approved these changes

Assignees

@julianolf julianolf

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove integration with Codecov Test workflow failing on Lint step
4 participants
@julianolf @gabriel-f-santos @Luis-Henrique @leonardoc-lima