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

Adding biome.package.json #233

Merged
merged 4 commits into from
Jul 15, 2024
Merged

Adding biome.package.json #233

merged 4 commits into from
Jul 15, 2024

Conversation

CarlosGamero
Copy link
Collaborator

@CarlosGamero CarlosGamero commented Jul 15, 2024

Changes

Please describe

Checklist

  • Apply one of following labels; major, minor, patch or skip-release
  • I've updated the documentation, or no changes were necessary
  • I've updated the tests, or no changes were necessary

@CarlosGamero CarlosGamero self-assigned this Jul 15, 2024
@@ -0,0 +1,11 @@
{
"$schema": "../../../node_modules/@biomejs/biome/configuration_schema.json",
"extends": ["./biome-base.jsonc"],
Copy link
Collaborator

Choose a reason for hiding this comment

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

unfortunately, extending from the same package is not supported by biome, it needs to be combined on a consumer side

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I was experiencing it right now 😓 then I guess that I should remove this remove on on package do

extends: [base, package]

right?

Copy link
Collaborator

Choose a reason for hiding this comment

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

yup!

@@ -0,0 +1,11 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

can you mention new preset in the documentation?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

On it, thank you!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

btw, is the following correct:


```json
{
	"$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
	"extends": ["./node_modules/@lokalise/biome-config/configs/biome-base.jsonc", "./node_modules/@lokalise/biome-config/configs/biome-esm.jsonc"]
}

I think we are not using esm config, right? asking just to understand it 🙏

Copy link
Collaborator

Choose a reason for hiding this comment

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

newer services (Billing, Workflows) use ESM, Autopilot doesn't (benefits of a switch do not currently justify the effort needed)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I see, thanks for explaining 🙏

@CarlosGamero
Copy link
Collaborator Author

Background-jobs-common had lint issues, they are not important and I don't think we need to release a new version of it so marking PR as skip-release and I will release new biome-config version manually

@CarlosGamero CarlosGamero marked this pull request as ready for review July 15, 2024 12:47
@CarlosGamero CarlosGamero requested review from dariacm, ondrejsevcik and a team as code owners July 15, 2024 12:47
@CarlosGamero CarlosGamero merged commit a57fe97 into main Jul 15, 2024
7 checks passed
@CarlosGamero CarlosGamero deleted the feature/biome-config-package branch July 15, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants