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

chore: publish some crates and add add workflow for publish crates #444

Merged
merged 7 commits into from
Oct 2, 2023

Conversation

nissy-dev
Copy link
Contributor

@nissy-dev nissy-dev commented Sep 28, 2023

Summary

Fix #190

I publish some crates and add add workflow for publish crates

see: https://crates.io/search?q=biome_

I invited @ematipico to each crates owner.

Test Plan

All tests should be passed

@github-actions github-actions bot added A-CLI Area: CLI A-Core Area: core A-Project Area: project A-Linter Area: linter A-Parser Area: parser A-Formatter Area: formatter A-Tooling Area: internal tools L-JavaScript Language: JavaScript and super languages L-CSS Language: CSS L-JSON Language: JSON and super languages A-Diagnostic Area: diagnostocis labels Sep 28, 2023
@nissy-dev

This comment was marked as outdated.

@@ -22,11 +21,11 @@ biome_rowan = { workspace = true }
tracing = { workspace = true }

[dev-dependencies]
biome_test_utils = { workspace = true }
biome_test_utils = { path = "../biome_test_utils" }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Use the path dependency to avoid requiring these crates to be published when doing cargo publish

@@ -83,7 +83,7 @@ quickcheck_macros = "1.0.0"
quote = { version = "1.0.28" }
rustc-hash = "1.1.0"
schemars = { version = "0.8.12" }
serde = { version = "1.0.163", features = ["derive"], default-features = false }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change is necessary to avioid build errors about some crates

Copy link
Member

@ematipico ematipico left a comment

Choose a reason for hiding this comment

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

That's great!

Maybe we should write somewhere that dev-dependencies must use the path

@nissy-dev
Copy link
Contributor Author

Thanks!

I'm going to edit contributing.md a bit after merging this, and I'll see if I can also add what you pointed out.

@nissy-dev nissy-dev merged commit af2b9bc into biomejs:main Oct 2, 2023
13 checks passed
@nissy-dev nissy-dev deleted the add-release-workflow branch November 14, 2023 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CLI Area: CLI A-Core Area: core A-Diagnostic Area: diagnostocis A-Formatter Area: formatter A-Linter Area: linter A-Parser Area: parser A-Project Area: project A-Tooling Area: internal tools L-CSS Language: CSS L-JavaScript Language: JavaScript and super languages L-JSON Language: JSON and super languages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants