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

Crate name refactor #561

Merged
merged 18 commits into from
Dec 13, 2023
Merged

Crate name refactor #561

merged 18 commits into from
Dec 13, 2023

Conversation

HCastano
Copy link
Collaborator

@HCastano HCastano commented Dec 13, 2023

This PR updates our crate names and folder structure a bit in preparation for publishing.

The main changes introduced are:

  • The crypto folder is now crates
  • Package names in crates are all now prefixed with entropy-
    • This will make it easier for us to get crate names on crates.io
  • The server binary has been renamed to entropy-tss
    • This name was too generic, and similarly we wouldn't be able to get this crate name
      on crates.io
    • Note that the published Docker images will now be entropyxyz/entropy-tss after
      this!

I hope this doesn't cause too much churn 😅

cc @vitropy @frankiebee @jawndiego

Copy link

vercel bot commented Dec 13, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
entropy-core ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 13, 2023 11:06pm

@@ -1,4 +1,6 @@
[profile.release]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We may want to add this back at some point, but actually populate it

Copy link
Contributor

@ameba23 ameba23 left a comment

Choose a reason for hiding this comment

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

I'm very happy to have entropy as a prefix rather than an acronym. And this is a big step towards being able to publish.

I guess the next thing is going to be trying to not have git dependencies...

@@ -122,7 +122,7 @@ jobs:
- release

parameters:
crypto:
crates:
Copy link
Contributor

Choose a reason for hiding this comment

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

Im happy with crates - but another possibilty would be tss-server or just tss since these crates are all related to or dependecies of entropy-tss.

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 wanted to keep it a bit generic since there are some crates used in both, and we could potentially have "standalone" crates that just happen to be part of the monorepo here

@HCastano HCastano merged commit f52d4d0 into master Dec 13, 2023
5 checks passed
@HCastano HCastano deleted the hc-rename-crates branch December 13, 2023 22:19
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.

3 participants