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

Support V2 API #8

Merged
merged 5 commits into from
Sep 2, 2022
Merged

Support V2 API #8

merged 5 commits into from
Sep 2, 2022

Conversation

adriansmares
Copy link

@adriansmares adriansmares commented Sep 2, 2022

Summary

References TheThingsNetwork/lorawan-stack#2798

This PR resets our fork of protoc-gen-validate to the latest version of the 0.5.1 tag, replays the migration to the V2 proto API from upstream, and then introduces back our non-gogo changes.

Changes

  • Upgrade to Go 1.18, Protoc 21.5 and google.golang.org/protobuf v1.28.1.
  • Upgrade to protoc-gen-star to v2.0.0-alpha1.

Torbjørn Vatn and others added 5 commits September 2, 2022 21:44
* Add support for module parameter

Signed-off-by: Torbjørn Vatn <torbjorn.vatn@unacast.com>

* Out path must be relative

Signed-off-by: Torbjørn Vatn <torbjorn.vatn@unacast.com>

* Add a short description to the README

Signed-off-by: Torbjørn Vatn <torbjorn.vatn@unacast.com>

* update other_package pkg and make to use module param

Signed-off-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>

* fix make clean

Signed-off-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>

Co-authored-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>
bufbuild#459)

* eliminates usage of old github.com/google/protobuf libraries, updates to google.golang.org/protobuf 1.5.2

Signed-off-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>

* fix remaining references

Signed-off-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>

* make gazelle

Signed-off-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>

* go mod tidy in tests

Signed-off-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>

* go mod tidy in tests

Signed-off-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>

* update Dockerfile

Signed-off-by: Matthew Crenshaw <mcrenshaw@clearstreet.io>
@adriansmares adriansmares self-assigned this Sep 2, 2022
@adriansmares adriansmares added this to the Q3 milestone Sep 2, 2022
@adriansmares adriansmares changed the title Feature/v2 api Support V2 API Sep 2, 2022
@adriansmares adriansmares merged commit 16c56b3 into develop Sep 2, 2022
@adriansmares adriansmares deleted the feature/v2-api branch September 2, 2022 21:13
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.

None yet

3 participants