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

Publish to crates.io #203

Closed
ascjones opened this issue Aug 31, 2023 · 3 comments · Fixed by #205
Closed

Publish to crates.io #203

ascjones opened this issue Aug 31, 2023 · 3 comments · Fixed by #205

Comments

@ascjones
Copy link
Contributor

Now that there are up to date crates released to crates.io, we should use those dependencies, and also publish substrate-contracts-node there.

We would need https://github.com/Brushfam/pallet-assets-chain-extension to be published to crates.io, or could copy it into this repository.

@deuszx
Copy link

deuszx commented Sep 1, 2023

Or we can remove the extension entirely from the subtrate-contracts-node. It seems like a random thing. Do we even know for sure that it's working correctly? There's no CI in the repo or tests.

@ascjones
Copy link
Contributor Author

ascjones commented Sep 1, 2023

#204

ascjones added a commit that referenced this issue Sep 1, 2023
Use all substrate/polkadot-sdk dependencies from crates.io, remove
`pallet-asset-chain-extension` so we can release this to crates.io. See
#203
ascjones added a commit that referenced this issue Sep 5, 2023
Closes #203. This will be the first release to crates.io:

- Use crates.io dependencies, remove chain extension #204
@ascjones
Copy link
Contributor Author

ascjones commented Sep 5, 2023

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 a pull request may close this issue.

2 participants