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

Updated grpc and google_protos deps. #17

Closed
wants to merge 1 commit into from

Conversation

wtcross
Copy link
Contributor

@wtcross wtcross commented Sep 6, 2023

The minor version of both grpc and google_protos have been bumped by one.
The client library was also regenerated using the buf cli.

The only file outside of generated code that was changed is mix.exs.

The minor version of both grpc and google_protos have been bumped
by one. The client library was also regenerated using the buf cli.

The only file outside of generated code that was changed is mix.exs.
@wtcross
Copy link
Contributor Author

wtcross commented Sep 6, 2023

I ran a local SpiceDB instance and verified that all tests are passing.

@wtcross
Copy link
Contributor Author

wtcross commented Sep 6, 2023

By the way, I'm very excited to start using SpiceDB with a large Elixir project. I am very grateful that you have created and are maintaining this library. I'll help out where I can once I really start digging into SpiceDB.

@wtcross
Copy link
Contributor Author

wtcross commented Sep 6, 2023

I just realized I forgot to update the version. I'll wait for feedback before doing that.

@goodhamgupta
Copy link
Owner

Hi @wtcross. Thank you for taking the time to improve this repo!

All your changes look good! Like you suggested, we can now add a version bump, and then I can merge the changes in!

@zagraves
Copy link

Hi @goodhamgupta. I'd love to start using a new version of authzed_ex with these changes made 🙏

@goodhamgupta
Copy link
Owner

Hi @zagraves and @wtcross, I've published the new version(0.0.6) to Hex today, and it's available here: https://hex.pm/packages/authzed/0.0.6

Apologies for the late response. I wasn't sure if the package version bump had been made since there was no activity on this PR. There were minor CI and formatting issues, so I forked this PR here: #18, and deployed this fork instead.

Thanks for your work @wtcross, and for your patience! 😄

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.

3 participants