Skip to content

Add build and lint workflows #33

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
Merged
basebandit merged 17 commits into from
May 16, 2025
Merged

Add build and lint workflows #33

basebandit merged 17 commits into from
May 16, 2025

Conversation

@basebandit
Copy link
Owner

@basebandit basebandit commented May 16, 2025

Description

  • Add gha workflows for build and lint

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

basebandit and others added 17 commits May 13, 2025 17:48
@basebandit
edit feature list markdown
cbb7819
@basebandit
add create pod handler and tool with unit tests
3f0db6c
@basebandit
update method signature
c95265d
@basebandit
add mocks for create pod behaviour
5a6d5d0
@basebandit
refactor to use table-driven tests
0d64ddf
@basebandit
Merge branch 'main' into playground-fix
22caf20
@basebandit
Merge pull request #28 from basebandit/playground-fix
9fa6cf0 
Refactor Pod tools Table tests
@basebandit
refactor to use table-driven tests
1c38d1f
@basebandit
Merge pull request #29 from basebandit/playground-fix
9a79c75 
Playground fix
@basebandit
add encodeing type
5416bcc
@basebandit
attempt to reduce sonar duplicate code metric count
f2c86a1
@basebandit
attempt to reduce sonar duplicate code metric count (#32)
5d64968
@basebandit
Merge branch 'main' into playground
499ab6b
@basebandit
attempt to reduce sonar duplicate code metric count
e5306a3
@basebandit
golang ci lint config
1fc79b4
@basebandit
add golang build and lint workflows
1f091fe
@basebandit
Trigger sonar scan to test ci analysis method (#34)
86415f9 
* add golang build and lint workflows

* fix lint workflow

* update lint rules and linters enabled
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 16, 2025

Please retry analysis of this Pull-Request directly on SonarQube Cloud

@sonarqubecloud
Copy link

sonarqubecloud bot commented May 16, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@basebandit basebandit merged commit ee4de35 into main May 16, 2025
10 of 14 checks passed
basebandit added a commit that referenced this pull request May 16, 2025
@basebandit
Add build and lint workflows (#33)
358c1bb 
* edit feature list markdown

* add create pod handler and tool with unit tests

* update method signature

* add mocks for create pod behaviour

* refactor to use table-driven tests

* refactor to use table-driven tests

* add encodeing type

* attempt to reduce sonar duplicate code  metric count

* attempt to reduce sonar duplicate code  metric count (#32)

* attempt to reduce sonar duplicate code  metric count

* golang ci lint config

* add golang build and lint workflows

* Trigger sonar scan to test ci analysis method (#34)

* add golang build and lint workflows

* fix lint workflow

* update lint rules and linters enabled
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.

2 participants

@basebandit