Skip to content
This repository has been archived by the owner on Jan 27, 2021. It is now read-only.

Fix protobuf gen #56

Merged
merged 2 commits into from
Aug 21, 2020
Merged

Fix protobuf gen #56

merged 2 commits into from
Aug 21, 2020

Conversation

kulmann
Copy link
Member

@kulmann kulmann commented Aug 21, 2020

Just checking... related to #30

@kulmann
Copy link
Member Author

kulmann commented Aug 21, 2020

@C0rby what do you think about this PR? 🤔

Copy link
Contributor

@C0rby C0rby left a comment

Choose a reason for hiding this comment

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

Looks good to me. Let's try, hopefully this will work! 👍

@kulmann
Copy link
Member Author

kulmann commented Aug 21, 2020

@C0rby I will restart CI on this PR like a hundred times. If there are no failures, I will merge.

@kulmann
Copy link
Member Author

kulmann commented Aug 21, 2020

@C0rby But you were right, the protoc related steps are not called in the make build step. So maybe triggering them explicitly helps... I still don't know why they get triggered sometimes... 🤷‍♂️

@C0rby
Copy link
Contributor

C0rby commented Aug 21, 2020

Yeah, thats so strange..

@kulmann
Copy link
Member Author

kulmann commented Aug 21, 2020

5 successful drone runs and 2 with github timeouts... merging now

@kulmann kulmann merged commit be5d9b8 into master Aug 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants