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

Various documentation improvements (#52600) #52623

Merged
merged 1 commit into from
May 9, 2024
Merged

Conversation

Hixie
Copy link
Contributor

@Hixie Hixie commented May 7, 2024

This is a reland of #52600 which was reverted in #52607 due to a typo.

Copy link
Member

@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!

@zanderso
Copy link
Member

zanderso commented May 9, 2024

From PR triage: @Hixie are you still working on this?

@Hixie Hixie added the autosubmit Merge PR when tree becomes green via auto submit App label May 9, 2024
@Hixie
Copy link
Contributor Author

Hixie commented May 9, 2024

yup, just didn't want to land this when i wasn't around :-)

@Hixie Hixie merged commit 6841af7 into flutter:main May 9, 2024
29 checks passed
@Hixie Hixie deleted the docs2 branch May 9, 2024 22:33
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 10, 2024
///
/// ```dart
/// // Matrix4's constructor arguments are in column-major order.
/// Matrix4(rst.scos, rst.ssin, 0, 0,
Copy link
Member

Choose a reason for hiding this comment

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

snippet analysis is sad that rst isn't initialized.

Copy link
Member

Choose a reason for hiding this comment

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

@zanderso
Copy link
Member

Reason for revert: Snippet analysis failure blocking the roll to the framework.

@zanderso zanderso added the revert Label used to revert changes in a closed and merged pull request. label May 10, 2024
@zanderso
Copy link
Member

The revert bot appears to be down. I'll kick off a manual revert.

zanderso added a commit that referenced this pull request May 10, 2024
zanderso added a commit that referenced this pull request May 10, 2024
Reverts #52623

Reason for revert is noted on the PR.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 10, 2024
zanderso pushed a commit to flutter/flutter that referenced this pull request May 10, 2024
…148137)

flutter/engine@c0917b1...1ccd0c3

2024-05-10 [email protected] Fixed constness
of display list storage. (flutter/engine#52705)
2024-05-10 [email protected] Revert "Various
documentation improvements (#52600)" (flutter/engine#52709)
2024-05-10 [email protected] Manual roll Dart SDK from
b7cad2edae4b to 01121c008f4d (3 revisions) (flutter/engine#52706)
2024-05-10 [email protected] [iOS] Fix App crash when use WebView with iOS
VoiceOver (flutter/engine#52484)
2024-05-09 [email protected] Various documentation improvements (#52600)
(flutter/engine#52623)
2024-05-09 [email protected] [web] scale semantic text elements to
match the desired focus ring size (flutter/engine#52586)
2024-05-09 [email protected] [Impeller] Adds
impeller display list golden tests (flutter/engine#52690)
2024-05-09 [email protected] Roll buildroot to
70a42312a688 (flutter/engine#52675)
2024-05-09 [email protected] When `et` is not attached
to a terminal, still split lines for status updates.
(flutter/engine#52681)
2024-05-09 [email protected] updated analysis
exclusion (flutter/engine#52699)

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] 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
Copy link
Contributor

auto-submit bot commented May 10, 2024

Unable to create the revert pull request due to ProcessException: Standard out
On branch revert_6841af712210d455b272216a1f1c11f33207042c
You are in a sparse checkout with 1% of tracked files present.

nothing to commit, working tree clean
Command: git revert --no-edit -m 1 6841af7

@auto-submit auto-submit bot removed the revert Label used to revert changes in a closed and merged pull request. label May 10, 2024
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 e: impeller
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants