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

Update Makefile and GitHub Actions Workflows #183

Merged
merged 1 commit into from Sep 4, 2022
Merged

Update Makefile and GitHub Actions Workflows #183

merged 1 commit into from Sep 4, 2022

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Sep 4, 2022

  • echo golangci-lint version
  • echo staticcheck version
  • echo markdownlint-cli version
  • add go_mod_changes job to test for uncommitted go mod tidy
    and go mod vendor changes
  • add CodeQL GitHub Actions Workflow

@atc0005
Update Makefile and GitHub Actions Workflows
21aaef4 
- echo golangci-lint version
- echo staticcheck version
- echo markdownlint-cli version
- add `go_mod_changes` job to test for uncommitted `go mod tidy`
  and `go mod vendor` changes
- add CodeQL GitHub Actions Workflow
@atc0005 atc0005 added enhancement New feature or request CI linting labels Sep 4, 2022
@atc0005 atc0005 added this to the v2.7.0 milestone Sep 4, 2022
@atc0005 atc0005 self-assigned this Sep 4, 2022
@atc0005
Copy link
Owner Author

atc0005 commented Sep 4, 2022

Note:

  • ignoring SA1019 linting error since this project (for now) lists compatibility with Go 1.14
  • ignoring File is not gofmt-ed with -s (gofmt) linting error (for now) since that is Go 1.19 specific

@atc0005 atc0005 merged commit 789ee25 into master Sep 4, 2022
@atc0005 atc0005 deleted the refresh-ci-and-makefile-workflows branch September 4, 2022 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@atc0005 atc0005

Labels
CI enhancement New feature or request linting
Projects
None yet
Milestone
v2.7.0
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@atc0005