Skip to content
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

⚡ Replace ESLint with Biome #346

Merged
merged 3 commits into from
Sep 20, 2024
Merged

⚡ Replace ESLint with Biome #346

merged 3 commits into from
Sep 20, 2024

Conversation

yannbertrand
Copy link
Contributor

Biome is way faster than ESLint which we almost don't use in this project. Adding it also provides a formater we can use with the npm run format command.

Kept defaults Biome config, lots of changes are due to tabs over spaces.

  • 🐛 Also fix Gitpod config (to use npm instead of yarn)
  • 👷 Also clean the GhA to optimize images (pin versions and formatting stuff)

@yannbertrand yannbertrand self-assigned this Sep 20, 2024
with:
title: Compressed Images Nightly
title: ':bento: Auto optimize images'
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
title: ':bento: Auto optimize images'
title: '🍱 Auto optimize images'

utf-8 for the win

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh what's the diff? I'm used to using Gitmoji names rather than the character. Should it render everywhere utf8 works?

Copy link
Contributor

Choose a reason for hiding this comment

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

emoji text doesn't render everything, you can use the same convention as gitmoji but with utf-8 unicode characters instead... then you don't have to teach your terminal to convert emoji text to utf-8 characters

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually GitHub supports them everywhere, that's why I thought it worked everywhere 😅

TIL, thanks!

.github/workflows/optimize-images.yml Outdated Show resolved Hide resolved
@yannbertrand yannbertrand merged commit be0c7bb into master Sep 20, 2024
3 checks passed
@yannbertrand yannbertrand deleted the reset-ducks branch September 20, 2024 15:35
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.

2 participants