-
Notifications
You must be signed in to change notification settings - Fork 4
Prettify, consolidate and deepwiki #6
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.
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
Conversation
…consistent formatting
…tion for code formatting
Co-authored-by: Copilot <[email protected]>
…nhance GithubAuth component with user prop for better context
|
@upayanmazumder is attempting to deploy a commit to the GDSC VIT's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request introduces comprehensive formatting and linting standards across the codebase by adding Prettier configuration and converting double quotes to single quotes throughout the project. The changes establish automated code formatting workflows and improve developer experience with consistent style enforcement.
Key Changes:
- Added comprehensive Prettier configuration with single quote enforcement and automated formatting
- Updated CI/CD workflows to include Prettier checks and Node.js dependency installation
- Converted all string literals from double quotes to single quotes across the entire codebase
Reviewed Changes
Copilot reviewed 159 out of 168 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| package.json | Added Prettier as dev dependency and formatting scripts |
| docker-compose.yml | Updated quotes to single quotes |
| app/tsconfig.json | Added composite flag and consolidated include array |
| Various .tsx/.ts files | Converted double quotes to single quotes for consistency |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
app/src/components/Applications/ConfigureApp/SetupTab/PortsSection/PortsSection.tsx
Outdated
Show resolved
Hide resolved
app/src/components/Applications/ConfigureApp/SetupTab/PortsSection/PortsSection.tsx
Outdated
Show resolved
Hide resolved
…them for improved subdomain extraction
This pull request introduces a comprehensive setup for code formatting and linting standards across the project, with a strong emphasis on enforcing Prettier formatting and consistent editor configurations. It also updates several files to use single quotes for string literals, aligning with the new formatting rules. The changes improve code consistency, automate formatting checks, and enhance developer experience.
Formatting and Linting Automation
.prettierrcfor formatting rules,.prettierignorefor ignored paths, and.lintstagedrc.jsonfor staged file formatting automation. [1] [2] [3].github/workflows/linter.ymlto install Node.js dependencies and run Prettier checks in CI, and set Super-Linter to use Prettier for JS/TS linting. [1] [2]Editor and Project Configuration
.editorconfigto enforce consistent indentation, line endings, and charset across editors..vscode/settings.jsonto use Prettier as the default formatter and enable format on save.Codebase Consistency
app/eslint.config.mjs,app/next.config.ts,app/public/service-worker.js,app/scripts/generate-runtime-env.js, and multiple files inapp/src/app/admin/). [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]Documentation
README.mdto add a DeepWiki badge and clarify dependency requirements. [1] [2]