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

fix(elixir): use local Rust extism package from extism_nif for local development #180

Merged
merged 2 commits into from
Dec 28, 2022

Conversation

zshipko
Copy link
Contributor

@zshipko zshipko commented Dec 19, 2022

No description provided.

@zshipko zshipko closed this Dec 19, 2022
@zshipko zshipko deleted the elixir-dev-use-local-sdk branch December 19, 2022 23:48
@zshipko zshipko restored the elixir-dev-use-local-sdk branch December 20, 2022 00:30
@zshipko zshipko reopened this Dec 20, 2022
@zshipko zshipko mentioned this pull request Dec 20, 2022
@@ -11,5 +11,5 @@ crate-type = ["cdylib"]

[dependencies]
rustler = "0.26.0"
extism = { version = "0.1.0" }
extism = { version = "0.1.0", path = "../../../rust" }
Copy link
Contributor

Choose a reason for hiding this comment

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

I had to remove this for a reason. Can't quite remember why. I think it was failing on release. I'll test it out tomorrow.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! Without this the changes from the local runtime crate aren’t be available, which tripped me up when I was testing something out.

Copy link
Member

Choose a reason for hiding this comment

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

@bhelx - did you get a chance to test this out? just don't want to let it sit too long

Copy link
Contributor

Choose a reason for hiding this comment

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

if i remember correctly, it was breaking on release. let me see if there is a way i can dry run it

Copy link
Contributor

Choose a reason for hiding this comment

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

mix hex.publish --dry-run worked

@bhelx bhelx merged commit 5c9aa4c into main Dec 28, 2022
@bhelx bhelx deleted the elixir-dev-use-local-sdk branch December 28, 2022 17:40
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 this pull request may close these issues.

None yet

3 participants