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

Decomposed schema crate. #30

Merged
merged 5 commits into from
Nov 9, 2023
Merged

Decomposed schema crate. #30

merged 5 commits into from
Nov 9, 2023

Conversation

pompon0
Copy link
Contributor

@pompon0 pompon0 commented Nov 8, 2023

Now that cross-crate proto imports are supported, we can move the proto files to crates which are actually using them.

@pompon0 pompon0 requested a review from brunoffranca as a code owner November 8, 2023 16:55
@pompon0 pompon0 requested a review from slowli November 8, 2023 16:55
brunoffranca
brunoffranca previously approved these changes Nov 8, 2023
Copy link
Member

@brunoffranca brunoffranca left a comment

Choose a reason for hiding this comment

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

At last!

@pompon0 pompon0 requested a review from brunoffranca November 9, 2023 09:04
@@ -23,12 +23,12 @@ env:

jobs:
compatibility:
runs-on: ubuntu-latest
runs-on: [ubuntu-22.04-github-hosted-16core]
Copy link
Contributor

Choose a reason for hiding this comment

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

Just to clarify: Is this change somehow related to decomposing the schema crate, or is it a general change that would be performed anyway?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this check is flaky on the default worker, now that a lot of stuff needs to get compiled.

@pompon0 pompon0 merged commit 3f27567 into main Nov 9, 2023
3 of 4 checks passed
@pompon0 pompon0 deleted the gprusak-split-schema branch November 9, 2023 10:11
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