Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@charafau
Copy link
Contributor

Link to engine setup was broken so here's the fix

CONTRIBUTING.md Outdated
[code_of_conduct]: https://github.com/flutter/flutter/blob/master/CODE_OF_CONDUCT.md
[contrib_guide]: https://github.com/flutter/flutter/blob/master/CONTRIBUTING.md
[engine_dev_setup]: https://github.com/flutter/flutter/blob/master/docs/engine/contributing/Setting-up-the-Engine-development-environment.md
[engine_dev_setup]: https://github.com/flutter/engine/blob/main/docs/contributing/Setting-up-the-Engine-development-environment.md
Copy link
Member

Choose a reason for hiding this comment

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

Nice! But, since this is now in the same repo, can this just be relative?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right, didn't know that it's a thing. Fixed, thank you!

@charafau charafau requested a review from chinmaygarde July 29, 2024 12:50
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 29, 2024
@auto-submit auto-submit bot merged commit a1243c7 into flutter:main Jul 29, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 30, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jul 30, 2024
…152524)

flutter/engine@49324cd...cb96743

2024-07-29 [email protected] [Impeller] delete rasterizer class. (flutter/engine#54196)
2024-07-29 [email protected] Roll Skia from 62e955dbaef0 to 832023dc041a (1 revision) (flutter/engine#54198)
2024-07-29 [email protected] [wiki] Fix link to engine dev setup (flutter/engine#54173)
2024-07-29 [email protected] Roll Dart SDK from 9a6e058303af to e8eca6f05494 (2 revisions) (flutter/engine#54197)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
TytaniumDev pushed a commit to TytaniumDev/flutter that referenced this pull request Aug 7, 2024
…lutter#152524)

flutter/engine@49324cd...cb96743

2024-07-29 [email protected] [Impeller] delete rasterizer class. (flutter/engine#54196)
2024-07-29 [email protected] Roll Skia from 62e955dbaef0 to 832023dc041a (1 revision) (flutter/engine#54198)
2024-07-29 [email protected] [wiki] Fix link to engine dev setup (flutter/engine#54173)
2024-07-29 [email protected] Roll Dart SDK from 9a6e058303af to e8eca6f05494 (2 revisions) (flutter/engine#54197)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Buchimi pushed a commit to Buchimi/flutter that referenced this pull request Sep 2, 2024
…lutter#152524)

flutter/engine@49324cd...cb96743

2024-07-29 [email protected] [Impeller] delete rasterizer class. (flutter/engine#54196)
2024-07-29 [email protected] Roll Skia from 62e955dbaef0 to 832023dc041a (1 revision) (flutter/engine#54198)
2024-07-29 [email protected] [wiki] Fix link to engine dev setup (flutter/engine#54173)
2024-07-29 [email protected] Roll Dart SDK from 9a6e058303af to e8eca6f05494 (2 revisions) (flutter/engine#54197)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants