Skip to content

ci/docs: Create CI pipeline for backend linting#36

Merged
SeanSan06 merged 8 commits intomainfrom
daniel/features/interview-questions
Mar 27, 2026
Merged

ci/docs: Create CI pipeline for backend linting#36
SeanSan06 merged 8 commits intomainfrom
daniel/features/interview-questions

Conversation

@drhhhh
Copy link
Copy Markdown
Collaborator

@drhhhh drhhhh commented Mar 24, 2026

Description

  • Adds 'backend-lint.yml' for automated python style and syntax checks using GitHub actions
  • Uses Black for code formatting
  • Uses Flake8 for syntax checks

Type of Change

  • New feature
  • Bug fix
  • Refactor
  • CI/CD update
  • Test
  • Other (please explain):

How to Test Changes

Ensure that the linting for Python works and shows up when a PR is made.

Checklist/Sanity Checking

  • I have run linting
  • I have run the build locally
  • I have tested my changes
  • I have updated documentation/comments if needed
  • I have updated the README if needed

- Added 'backend-lint.yml' for automated python style and syntax checks
    using github actions
- Uses Black for code formatting
- Uses Flake8 for syntax checks
@drhhhh drhhhh requested a review from elcosas as a code owner March 24, 2026 00:03
@github-actions github-actions bot added the ci label Mar 24, 2026
@SeanSan06 SeanSan06 requested review from SeanSan06 March 24, 2026 01:58
@SeanSan06
Copy link
Copy Markdown
Owner

@drhhhh I will review this code.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 24, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
the-method Ignored Ignored Preview Mar 25, 2026 6:14am

Before it would only run when a change is made in the backend folder.
Now it will run on all pushes to the repo.
drhhhh added 2 commits March 24, 2026 12:57
Reformatted all Python files in the backend directory using Black
to ensure consistent code style.
changes made to backend linting workflow
@drhhhh drhhhh requested a review from vedevpatel as a code owner March 24, 2026 20:07
@SeanSan06
Copy link
Copy Markdown
Owner

@drhhhh This is good, I will review your formatting changes, write a review, and merged the code later.

This requirements-dev.txt file is just for developers and does not need
to be added as a command in the docker containers.
@SeanSan06 SeanSan06 changed the title Create CI pipeline for backend linting chore: Create CI pipeline for backend linting Mar 27, 2026
@SeanSan06 SeanSan06 changed the title chore: Create CI pipeline for backend linting ci/docs: Create CI pipeline for backend linting Mar 27, 2026
Copy link
Copy Markdown
Owner

@SeanSan06 SeanSan06 left a comment

Choose a reason for hiding this comment

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

Everything looks good. The CI for linting the Python code works. The README.md for the backend has also been update accordingly.

@SeanSan06 SeanSan06 merged commit 49462c3 into main Mar 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants