Skip to content

Add linting GitHub action - #124

Merged
ericguo202 merged 9 commits into
previewfrom
add-linting-github-action
Sep 8, 2026
Merged

ericguo202 merged 9 commits into
previewfrom
add-linting-github-action

Conversation

@issacli-0821

Copy link
Copy Markdown
Contributor

References

Proposed Changes

  • Add linting to Github workflow
  • Fix all preexsiting linting warnings for backend and frontend

@vercel

vercel Bot commented Apr 23, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
help Ready Ready Preview Sep 8, 2026 3:53am UTC

Request Review

- Use `uv sync --locked` so a stale uv.lock fails CI instead of being
  silently re-resolved, matching `yarn install --immutable` on the frontend.
- Pin ruff to 0.15.11 (the version already in uv.lock). Formatter output
  changes between ruff minors, so an unpinned lower bound could red the
  build on untouched files after a lock refresh.
- Add a concurrency group so force-pushes cancel superseded runs.
- Add the missing trailing newline in lint.yml.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TfhF63MKXTpcgcvSfFNFiq

@ericguo202 ericguo202 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor bugs/issues fixed

@ericguo202
ericguo202 merged commit 2ada27e into preview Sep 8, 2026
4 checks passed
@ericguo202
ericguo202 deleted the add-linting-github-action branch October 11, 2026 17:32

This branch was successfully deployed

1 active deployment
Preview — c75ab3b0 Deployed Sep 8, 2026 by vercel[bot]
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.

2 participants