Skip to content

Conversation

@JonathanMatthey
Copy link
Contributor

Changes:

  • Added ESLint dependencies (eslint, @typescript-eslint/*, eslint-plugin-astro, astro-eslint-parser)
  • Created .eslintrc.cjs with TypeScript and Astro support
  • Added scripts: lint, lint:fix, format
  • Created .eslintignore to exclude build artifacts and dependencies
  • Updated README with formatting setup and usage instructions
  • Added/updated PR template with formatting checklist
  • Added CI workflow to enforce ESLint checks on PRs

@JonathanMatthey JonathanMatthey self-assigned this Nov 17, 2025
@JonathanMatthey JonathanMatthey force-pushed the jm/eslint-formatting branch 3 times, most recently from 03cd285 to 7fb064f Compare November 20, 2025 18:45
…ards

- Add ESLint 9 and Prettier for consistent code formatting across docs repos
- Configure ESLint with TypeScript and Astro support
- Standardize lint and format commands across all repositories
- Upgrade to ESLint 9 flat config format (eslint.config.js)
- Update all ESLint and Prettier packages to latest versions
- Add CI checks to enforce formatting standards
Copy link
Contributor

@Anca2022 Anca2022 left a comment

Choose a reason for hiding this comment

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

Consistency changes

| Command | Action |
| :--------------- | :----------------- |
| `bun run format` | Fix linting issues |
| `bun run lint` | Check linting |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| `bun run lint` | Check linting |
| `bun run lint` | Check code formatting and linting |


| Command | Action |
| :--------------- | :----------------- |
| `bun run format` | Fix linting issues |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| `bun run format` | Fix linting issues |
| `bun run format` | Format code and fix linting issues |

- uses: oven-sh/setup-bun@v1
- name: Install dependencies
run: bun install
- name: Run ESLint
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- name: Run ESLint
- name: Check formatting and linting


### 🔍 Code Linting

This project uses [ESLint](https://eslint.org/) for code linting. Before submitting a pull request, please ensure your code passes linting:
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add prettier and Biome in here

@JonathanMatthey JonathanMatthey deleted the jm/eslint-formatting branch November 28, 2025 20:13
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.

3 participants