Skip to content

Add golangci-lint GitHub Action workflow #4945

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
Draft
dplore wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Add golangci-lint GitHub Action workflow #4945

dplore wants to merge 8 commits into from

Conversation

@dplore
Copy link
Member

@dplore dplore commented Dec 19, 2025

No description provided.

@gemini-code-assist
Copy link
Contributor

gemini-code-assist bot commented Dec 19, 2025

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@OpenConfigBot
Copy link

OpenConfigBot commented Dec 19, 2025 •
edited
Loading

Pull Request Functional Test Report for #4945 / 86c5c30

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
 System-1.3: System hostname test
 ef37f1c8 Log
Cisco 8000E status
 System-1.3: System hostname test
 9727603f Log
Cisco XRd status
 System-1.3: System hostname test
 b6919154 Log
Juniper ncPTX status
 System-1.3: System hostname test
 0d52678b Log
Nokia SR Linux status
 System-1.3: System hostname test
 b1a0d52b Log
Openconfig Lemming status
 System-1.3: System hostname test
 92852251 Log

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
 System-1.3: System hostname test
Cisco 8808 status
 System-1.3: System hostname test
Juniper PTX10008 status
 System-1.3: System hostname test
Nokia 7250 IXR-10e status
 System-1.3: System hostname test

Help

@dplore
add golangci config
2c9bb86
@dplore
Update hostname test case with new hostname
b14e737
@dplore
Refactor golangci-lint workflow for static analysis
c0bd1f1 
Updated the GitHub Actions workflow for static analysis, adding steps for staticcheck and modifying the golangci-lint step.
@dplore
Rename workflow to 'New Static Analysis'
7c87e9d
github-advanced-security[bot]
github-advanced-security bot found potential problems Dec 19, 2025
View reviewed changes
@coveralls
Copy link

coveralls commented Dec 19, 2025

Pull Request Test Coverage Report for Build 20360771703

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.034%
Totals Coverage Status
Change from base Build 20355399115: 0.0%
Covered Lines: 2227
Relevant Lines: 22195
💛 - Coveralls

@coveralls
Copy link

coveralls commented Dec 19, 2025

Pull Request Test Coverage Report for Build 20361042851

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.034%
Totals Coverage Status
Change from base Build 20355399115: 0.0%
Covered Lines: 2227
Relevant Lines: 22195
💛 - Coveralls

@coveralls
Copy link

coveralls commented Dec 19, 2025

Pull Request Test Coverage Report for Build 20360829433

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.034%
Totals Coverage Status
Change from base Build 20355399115: 0.0%
Covered Lines: 2227
Relevant Lines: 22195
💛 - Coveralls

@coveralls
Copy link

coveralls commented Dec 19, 2025

Pull Request Test Coverage Report for Build 20360965084

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.034%
Totals Coverage Status
Change from base Build 20355399115: 0.0%
Covered Lines: 2227
Relevant Lines: 22195
💛 - Coveralls

@coveralls
Copy link

coveralls commented Dec 19, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build 20382453315

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.034%
Totals Coverage Status
Change from base Build 20355399115: 0.0%
Covered Lines: 2227
Relevant Lines: 22195
💛 - Coveralls

@coveralls
Copy link

coveralls commented Dec 19, 2025

Pull Request Test Coverage Report for Build 20360869823

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.034%
Totals Coverage Status
Change from base Build 20355399115: 0.0%
Covered Lines: 2227
Relevant Lines: 22195
💛 - Coveralls

@coveralls
Copy link

coveralls commented Dec 19, 2025

Pull Request Test Coverage Report for Build 20360961839

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.034%
Totals Coverage Status
Change from base Build 20355399115: 0.0%
Covered Lines: 2227
Relevant Lines: 22195
💛 - Coveralls

@dplore
Add step to free disk space in CI workflow
86c5c30 
Added a step to free additional disk space before setting up Go.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dplore @OpenConfigBot @coveralls