chore: format with Biome #79
Conversation
|
| @@ -555,8 +555,8 @@ export default function createIntegration(args?: Options): AstroIntegration { | |||
| const winningStrategy = notFoundIsSSR | |||
| ? excludeStrategy | |||
| : includeStrategyLength <= excludeStrategyLength | |||
| ? includeStrategy | |||
| : excludeStrategy; | |||
| ? includeStrategy | |||
There was a problem hiding this comment.
EDIT: this is the new formatting of ternaries in Prettier 3.1
| "ignore": [ | ||
| "**/dist", | ||
| "**/node_modules", | ||
| "**/fixtures", |
There was a problem hiding this comment.
Is there a reason why we ignore the fixtures?
There was a problem hiding this comment.
@ematipico there was a reason, but I can't remember it.
I'm happy to enable it for fixtures :)
ematipico
force-pushed
the
chore/format-with-biome
branch
2 times, most recently
from
40c52f7 to
a554989
Compare
| .replace(/[🚀┃]/g, '') | ||
| .replace(/\s+/g, ' ') | ||
| .trim() | ||
| ); | ||
|
|
There was a problem hiding this comment.
nit: I personally dislike that it is not formatted like:
.map((ln) =>
ln.replace(/[🚀┃]/g, '').replace(/\s+/g, ' ').trim()
);There was a problem hiding this comment.
It's a bug, I will report it back in the Biome repository. Also, I think we have more fixes in the pipeline coming in the next release.
ematipico
force-pushed
the
chore/format-with-biome
branch
from
a554989 to
314f033
Compare
Changes
This PR makes use of Biome to format the code
Testing
The current CI should pass
Docs
N/A