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

Revert "chore: Update Proto + ~~ORM Support~~ (#1589)" #2614

Merged
merged 8 commits into from
Sep 13, 2022

Conversation

mattverse
Copy link
Member

This reverts commit f291e12.

Some of the changes that were made in #1589 were changes that we wanted to keep(how we use buf gen to protogen, for example), making the scope of this PR not only simply revert the commit, but keep some of the updated process for proto gen that were initially included in #1589

@mattverse mattverse requested a review from a team September 6, 2022 08:07
@mattverse mattverse added the A:backport/v12.x backport patches to v12.x branch label Sep 8, 2022
scripts/protocgen.sh Outdated Show resolved Hide resolved
@mattverse mattverse merged commit 79adab9 into main Sep 13, 2022
@mattverse mattverse deleted the mattverse/orm-revert branch September 13, 2022 07:10
mergify bot pushed a commit that referenced this pull request Sep 13, 2022
* Revert "chore: Update Proto + ~~ORM Support~~ (#1589)"

This reverts commit f291e12.

* Update scripts/protocgen.sh

Co-authored-by: Roman <[email protected]>

* Run buf mod update

* query gen

* Change pb

Co-authored-by: Roman <[email protected]>
(cherry picked from commit 79adab9)
p0mvn pushed a commit that referenced this pull request Sep 14, 2022
… (#2720)

* Revert "chore: Update Proto + ~~ORM Support~~ (#1589)" (#2614)

* Revert "chore: Update Proto + ~~ORM Support~~ (#1589)"

This reverts commit f291e12.

* Update scripts/protocgen.sh

Co-authored-by: Roman <[email protected]>

* Run buf mod update

* query gen

* Change pb

Co-authored-by: Roman <[email protected]>
(cherry picked from commit 79adab9)

* Fix proto check

Co-authored-by: Matt, Park <[email protected]>
Co-authored-by: mattverse <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:backport/v12.x backport patches to v12.x branch C:CLI C:x/twap Changes to the twap module T:build
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants