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

Remove superfluous INFO logs from //flutter/runtime.#16280

Merged
chinmaygarde merged 1 commit into
flutter-team-archive:masterfrom
chinmaygarde:better_logs
Jan 31, 2020
Merged

Remove superfluous INFO logs from //flutter/runtime.#16280
chinmaygarde merged 1 commit into
flutter-team-archive:masterfrom
chinmaygarde:better_logs

Conversation

@chinmaygarde

Copy link
Copy Markdown
Contributor

These add no value to engine developers anymore and are not visible to external
users because of the low log severity.

These add no value to engine developers anymore and are not visible to external
users because of the low log severity.
@flar

flar commented Jan 31, 2020

Copy link
Copy Markdown
Contributor

No objections.

@chinmaygarde

Copy link
Copy Markdown
Contributor Author

No objections.

Can you press the lgtm button :)

@chinmaygarde chinmaygarde merged commit d6b75b7 into flutter-team-archive:master Jan 31, 2020
@chinmaygarde chinmaygarde deleted the better_logs branch January 31, 2020 02:53
@flar

flar commented Jan 31, 2020

Copy link
Copy Markdown
Contributor

No objections.

Can you press the lgtm button :)

I left it to Jason to approve as I'm not familiar with this code.

engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 31, 2020
flutter-team-archive/engine@804dca6...6b8c150

git log 804dca6..6b8c150 --first-parent --oneline
2020-01-31 gw280@google.com Use static constexpr for iphone_xs_times (flutter-team-archive/engine#16264)
2020-01-31 30870216+gaaclarke@users.noreply.github.com Started coalescing updates to viewport metrics after foregrounding the app. (flutter-team-archive/engine#16270)
2020-01-31 chinmaygarde@google.com Remove superfluous INFO logs from //flutter/runtime. (flutter-team-archive/engine#16280)


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 cbracken@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
…chive#16280)

These add no value to engine developers anymore and are not visible to external
users because of the low log severity.
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

4 participants