Set Up Linting, Formatting, and Pre-commit Hooks for Backend Codebase #236
Conversation
|
@medss19 is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
|
@medss19 this has been failed |
- Fix setup-node action failing due to missing package-lock.json
- Generate package-lock.json with --legacy-peer-deps for Firebase compatibility
- Update CI workflow to use 'npm install --legacy-peer-deps' instead of 'npm ci'
- Simplify linting scope to JS/JSX files only for better reliability
- Update lint-staged configuration to process *.{js,jsx} files only
- Update npm scripts for format/format:check to target JS/JSX files
- Format existing JavaScript files with Prettier
- Enable CI pipeline caching and proper dependency resolution
Resolves: Dependencies lock file not found error in GitHub Actions
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
medss19
force-pushed
the
linting
branch
2 times, most recently
from
a84fa5f to
e431895
Compare
Description
Fixes #121
Provide a brief summary of the changes made to the website and the motivation behind them. Include any relevant issues or tickets.
This helps fast tracking your PR and merge it, Check the respective box below.
Fixes # (issue)
Type of Change
Checklist
npm run buildand attached scrrenshot in this PR.Summary
This PR introduces a complete linting and formatting pipeline for the backend Python codebase to ensure consistent code quality and style across contributions.
Included Tools
flake8– Linting and PEP8 checksblack– Auto code formatterisort– Import sortermypy– Static type checkerpre-commit– Local enforcement of rulesWhat's Done
.flake8,pyproject.toml, andmypy.inifor configurations.pre-commit-config.yamlwith hooks forblack,flake8,isort, andmypyrequirements-dev.txtfor development dependencies.github/workflows/lint.ymlto run checks on PRsREADME.mdwith setup and usage instructionsBenefits