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

Empty definition from google #204

Closed
wants to merge 4 commits into from
Closed

Conversation

didrocks
Copy link
Collaborator

Instead of using our own, use the empty google message in the predefined repository.

Instead of using our own, use the google one in the predefined
repository.
Dart code was generated with 3.7.12.
@didrocks didrocks force-pushed the empty-definition-from-google branch 2 times, most recently from 119b3c2 to 8e2b019 Compare July 31, 2023 09:16
@didrocks didrocks force-pushed the empty-definition-from-google branch from 8e2b019 to 9d3f69d Compare July 31, 2023 09:19
@didrocks
Copy link
Collaborator Author

Ok, so there are some CI jobs failing related to Flutter. I have regenerated the binding and tried to do the API change aligning Flutter version (3.7.12).
I did try to run build_runner build (which requires a newer version than the one set in CI btw due to Dart 3.0), but this introduces quite some diff, including format change in gui/packages/ubuntupro/pubspec.lock. As I think you will have more expert eyes than I on this, I would appreciate if you can take a look @CarlosNihelton (feel free to directly amend/push this branch).

@didrocks
Copy link
Collaborator Author

didrocks commented Aug 1, 2023

As discussed on ubuntu/adsys#750, this actually has more drawbacks than advantages. Let’s keep decoupled from the google repo as long as we don’t use other types of messages. Withdrawing then.

@didrocks didrocks closed this Aug 1, 2023
@CarlosNihelton CarlosNihelton deleted the empty-definition-from-google branch September 18, 2024 19:25
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.

1 participant