8000 feat: Add golangci-lint & github action CI (#80) by Jeongseup · Pull Request #86 · cosmostation/cvms · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: Add golangci-lint & github action CI (#80) #86

New iss 8000 ue

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

Merged
merged 2 commits into from
May 23, 2025

Conversation

Jeongseup
Copy link
Member
@Jeongseup Jeongseup commented May 23, 2025
  • Add .golangci.yml

  • Add lint docs for dev

  • Fix whitespace lint issue

  • Fix dupl lint issue

  • Add cursor rule for open source dev

  • Remove cursor rule

  • Add ci.yml

  • Bump golangci-lint-action version

  • Update ci.yml for build docker image

  • Update ci.yml

  • Update ci workflow for CVMS

PR(Pull Request) Overview

Briefly describe the changes made in this PR.

Changes

  • Major feature addition or modification
  • Bug fix
  • Code improvement
  • Documentation update

Related Issue

Please reference the Issue number this PR addresses: #123

Description of Changes

Describe the key changes or parts of the code that have been modified.

Testing Method

Explain how to test the changes step by step: 1. Run ‘…’ 2. Check ‘…’

Additional Information

If there is any additional information relevant to this PR, please include it here.

* Add .golangci.yml

* Add lint docs for dev

* Fix whitespace lint issue

* Fix dupl lint issue

* Add cursor rule for open source dev

* Remove cursor rule

* Add ci.yml

* Bump golangci-lint-action version

* Update ci.yml for build docker image

* Update ci.yml

* Update ci workflow for CVMS
@Jeongseup Jeongseup requested a review from BOSEES May 23, 2025 02:51
@Jeongseup Jeongseup merged commit 6b8f093 into develop May 23, 2025
2 checks passed
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.

1 participant
0