Lint and format #135
Open
Lint and format #135
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
IEvangelist
requested review from
JamesNK,
aaronpowell,
captainsafia,
eerhardt and
mitchdenny
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request adds linting and formatting infrastructure to the frontend project, switches the package manager from npm to pnpm, and performs an initial formatting run across the codebase. The changes primarily consist of code style improvements and configuration updates to establish consistent formatting standards.
Key Changes
- Switched package manager from npm to pnpm
- Added and configured ESLint/Prettier for code formatting
- Performed initial formatting run on JavaScript, TypeScript, CSS, MDX, and JSON files
- Added custom Prettier plugin for MDX
<Steps>component handling
Reviewed changes
Copilot reviewed 192 out of 234 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
src/frontend/tsconfig.json |
Removed trailing comma from paths configuration |
src/frontend/src/utils/imageImporter.js |
Formatted import statement and arrow function syntax |
src/frontend/src/utils/helpers.js |
Standardized quote usage from double to single quotes |
src/frontend/src/styles/site.css |
Normalized hex color codes to lowercase, standardized indentation and quote usage |
src/frontend/src/styles/cookieconsent-custom.css |
Normalized hex color codes to lowercase, standardized indentation |
src/frontend/src/prettier-plugins/prettier-plugin-starlight-steps.mjs |
Added new Prettier plugin for MDX Steps component |
src/frontend/src/pages/rss.xml.js |
Formatted multi-line expressions and standardized quotes |
src/frontend/src/pages/1ds.js |
Added trailing commas and standardized arrow function syntax |
src/frontend/src/expressive-code-plugins/disable-copy.mjs |
Fixed indentation in nested braces |
src/frontend/src/data/aspire-posts.json |
Collapsed single-item arrays to single lines |
src/frontend/src/content/docs/**/*.mdx |
Added blank lines within <Steps> components, reformatted long lines in <Aside> components, fixed table formatting, standardized component prop formatting |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
… and astro for now.
- Removed "proseWrap" setting from Prettier configuration. - Added spacing in various documentation sections for improved readability. - Updated Azure integration documentation to include steps for connecting to storage resources. - Enhanced migration documentation with clearer steps for applying migrations. - Added settings for Prettier in VSCode to support Astro files.
IEvangelist
force-pushed
the
lint-and-format
branch
from
0d0c381 to
b11c33c
Compare
…; adjust contributor guide for simplified usage
…ns for improved functionality
…ependency management
…d adjust VSCode extensions
davidfowl
approved these changes
Dec 9, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add lint/format bits, switch to
pnpm, and perform initial run. Fixes #118