Skip to content

feat: add linter to CI#14

Merged
nomeguy merged 5 commits intomasterfrom
copilot/add-linter-ci-scripts
Jan 24, 2026
Merged

feat: add linter to CI#14
nomeguy merged 5 commits intomasterfrom
copilot/add-linter-ci-scripts

Conversation

Copy link
Contributor

Copilot AI commented Jan 18, 2026 •
edited by nomeguy
Loading

fix: #13

@CLAassistant
Copy link

CLAassistant commented Jan 18, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI and others added 3 commits January 18, 2026 13:22
@nomeguy
Add CI workflow for backend, frontend linter and build scripts
6458dc8 
Co-authored-by: nomeguy <85475922+nomeguy@users.noreply.github.com>
@nomeguy
Fix Node.js version to be a string in workflow
29a5a75 
Co-authored-by: nomeguy <85475922+nomeguy@users.noreply.github.com>
@nomeguy
Add explicit permissions block to limit GITHUB_TOKEN
ba7e2b5 
Co-authored-by: nomeguy <85475922+nomeguy@users.noreply.github.com>
Copilot AI changed the title [WIP] Add backend and frontend linter CI scripts Add CI workflows for backend/frontend builds and linting Jan 18, 2026
Copilot AI requested a review from nomeguy January 18, 2026 13:26
@nomeguy nomeguy marked this pull request as ready for review January 18, 2026 16:16
@nomeguy
Copy link
Contributor

nomeguy commented Jan 24, 2026

@copilot fix failed CI check: https://github.com/casbin/policyguard/actions/runs/21112473205?pr=14

<html>
<body>
<!--StartFragment--><div class="mt-1 mb-3 px-3" style="box-sizing: border-box; margin-top: 4px !important; margin-bottom: 16px !important; padding-right: 16px !important; padding-left: 16px !important; color: rgb(31, 35, 40); font-family: -apple-system, BlinkMacSystemFont, &quot;Segoe UI&quot;, &quot;Noto Sans&quot;, Helvetica, Arial, sans-serif, &quot;Apple Color Emoji&quot;, &quot;Segoe UI Emoji&quot;; font-size: 14px; font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; white-space: normal; text-decoration-thickness: initial; text-decoration-style: initial; text-decoration-color: initial;"><div class="mx-0 mx-md-1" style="box-sizing: border-box; margin-right: 4px !important; margin-left: 4px !important;"><h2 class="text-bold h4" style="box-sizing: border-box; margin-top: 0px; margin-bottom: 0px; font-size: 16px !important; font-weight: 600 !important;">Annotations</h2><div class="color-fg-muted text-small" style="box-sizing: border-box; color: rgb(89, 99, 110) !important; font-size: 12px !important;">7 errors and 2 warnings</div></div></div>
Go-Linterissues found
--
Go-Linter: util/string.go#L107File is not properly formatted (gofumpt)
Go-Linter: util/string.go#L91File is not properly formatted (gofumpt)
Go-Linter: util/json.go#L7File is not properly formatted (gofumpt)
Go-Linter: routers/router.go#L14File is not properly formatted (gofumpt)
Go-Linter: object/dataset.go#L88File is not properly formatted (gofumpt)
Go-Linter: main.go#L21File is not properly formatted (gofumpt)

<!--EndFragment-->
</body>
</html>Annotations
7 errors and 2 warnings
[Go-Linter](https://github.com/casbin/policyguard/actions/runs/21112473205/job/60718734868#step:5:36)
issues found
[Go-Linter: util/string.go#L107](https://github.com/casbin/policyguard/pull/14/files#annotation_44307639879)
File is not properly formatted (gofumpt)
[Go-Linter: util/string.go#L91](https://github.com/casbin/policyguard/pull/14/files#annotation_44307639882)
File is not properly formatted (gofumpt)
[Go-Linter: util/json.go#L7](https://github.com/casbin/policyguard/pull/14/files#annotation_44307639885)
File is not properly formatted (gofumpt)
[Go-Linter: routers/router.go#L14](https://github.com/casbin/policyguard/pull/14/files#annotation_44307639889)
File is not properly formatted (gofumpt)
[Go-Linter: object/dataset.go#L88](https://github.com/casbin/policyguard/pull/14/files#annotation_44307639890)
File is not properly formatted (gofumpt)
[Go-Linter: main.go#L21](https://github.com/casbin/policyguard/pull/14/files#annotation_44307639894)
File is not properly formatted (gofumpt)

@nomeguy
Fix gofumpt formatting issues in Go files
03e7763 
Co-authored-by: nomeguy <85475922+nomeguy@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 24, 2026

@copilot fix failed CI check: https://github.com/casbin/policyguard/actions/runs/21112473205?pr=14

...

Fixed in commit 03e7763. Ran gofumpt -w on all affected files to resolve the formatting issues:

  • util/string.go: Updated octal permissions to use 0o644 notation
  • util/json.go, object/dataset.go, main.go: Added spaces after // in comments
  • routers/router.go: Reformatted multi-line variable assignment

All files now pass gofumpt validation.

@nomeguy nomeguy changed the title Add CI workflows for backend/frontend builds and linting feat: add linter to CI Jan 24, 2026
@nomeguy nomeguy merged commit b8ad4f2 into master Jan 24, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nomeguy nomeguy Awaiting requested review from nomeguy

Assignees

@nomeguy nomeguy

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[feature] add backend, frontend linter CI scripts

3 participants

@CLAassistant @nomeguy