-
Notifications
You must be signed in to change notification settings - Fork 28k
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
Configure flutter/impeller with all our CI automation #97686
Comments
This would make a good use of the go/flutter-infra-office-hours (Google internal link). The tl;dr from the infra side is:
|
|
It looks like the scorecard check started failing in impeller starting on one of my commits yesterday, but I'm not sure how to diagnose the problem. Is there a different dashboard that has more diagnostic/actionable output? I'm not seeing any here: https://github.com/flutter/impeller/runs/5327601854?check_suite_focus=true |
That's odd. Can you file a new infra bug for that? |
Sure, done: #99185 |
Before we jump into this, I think we first need to decide whether to simply bring the impeller source into the Engine repo directly instead of having a separate repo. We might discuss other motivations, but it's only in a separate repo now to help us bootstrap. |
I'll figure out the licenses stuff this week. In the engine sync last week, we discussed moving the repo entirely into the engine repo but decided against it at this time. Instead, re-evaluating the decision on a week by week basis. |
Sounds good, please let us know if it is decided to keep the repo to enable CI on it. |
Matches the one in the engine exactly as described in the [umbrella issue](flutter/flutter#97686).
Matches the one in the engine exactly as described in the [umbrella issue](flutter/flutter#97686).
…lock. Also fixes the files with missing licenses. This check is somewhat easy with Impeller than in the engine because all source files must have the same license block. Resolves an action item in the umbrella issue flutter/flutter#97686
…lock. (#63) Also fixes the files with missing licenses. This check is somewhat easy with Impeller than in the engine because all source files must have the same license block. Resolves an action item in the umbrella issue flutter/flutter#97686.
I'll ping this thread as we re-evaluate. For now, it is a separate repo. I've completed the action items related to licenses. |
At the meeting last week, we decided to re-evaluate again this week. |
Moving to P4 until we get more clarity on whether we will be enabling CI on github/impeller or not. |
Matches the one in the engine exactly as described in the [umbrella issue](flutter/flutter#97686).
…lock. (#63) Also fixes the files with missing licenses. This check is somewhat easy with Impeller than in the engine because all source files must have the same license block. Resolves an action item in the umbrella issue flutter/flutter#97686.
Matches the one in the engine exactly as described in the [umbrella issue](flutter/flutter#97686).
…lock. (#63) Also fixes the files with missing licenses. This check is somewhat easy with Impeller than in the engine because all source files must have the same license block. Resolves an action item in the umbrella issue flutter/flutter#97686.
This is obsolete. Impeller has moved into the Engine repo. |
This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of |
We just created flutter/impeller: https://github.com/flutter/impeller
We need to:
The text was updated successfully, but these errors were encountered: