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

Conversation

@camsim99
Copy link
Contributor

@camsim99 camsim99 commented Feb 14, 2022

This PR adds a compiler option to show the deprecation warnings when building the Android engine and a reminder to make these warnings fatal once the current ones are fixed.

As per this comment, I can confirm that this does not affect running flutter run/flutter create.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

@camsim99 camsim99 requested a review from blasten February 14, 2022 19:14
'-classpath', ':'.join(classpath),
'-d', classes_dir]
'-d', classes_dir,
# TODO(camsim99): Make deprecation warnings fatal and remove limit
Copy link
Member

Choose a reason for hiding this comment

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

This needs to be filed as as bug and the number cross-referenced here.

Copy link

@blasten blasten left a comment

Choose a reason for hiding this comment

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

LGTM

@camsim99 camsim99 merged commit 10e720f into flutter:master Feb 18, 2022
bdero added a commit that referenced this pull request Feb 23, 2022
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.

3 participants