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

Add missing inputs declaration#40133

Merged
jason-simmons merged 1 commit intoflutter:mainfrom
eseidel:patch-1
Mar 8, 2023
Merged

Add missing inputs declaration#40133
jason-simmons merged 1 commit intoflutter:mainfrom
eseidel:patch-1

Conversation

@eseidel
Copy link
Contributor

@eseidel eseidel commented Mar 8, 2023

inputs = [ "android_exports.lst" ]
was missing from the flutter_shell_native rule, which meant if you edited android_exports.lst it would not notice and would not rebuild/relink.

Pre-launch Checklist

  • [x ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • [ x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • [ x] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides].
  • I listed at least one issue that this PR fixes in the description above.
  • [x ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See [testing the engine] for instructions on writing and running engine tests.
  • [x ] I updated/added relevant documentation (doc comments with ///).
  • [ x] I signed the [CLA].
  • [ x] All existing and new tests are passing.

  inputs = [ "android_exports.lst" ]
was missing from the flutter_shell_native rule, which meant if you edited android_exports.lst it would not notice and would not rebuild/relink.
@eseidel
Copy link
Contributor Author

eseidel commented Mar 8, 2023

I don't believe I have the ability to land. If someone could put this in the submit queue for me, that'd be awesome. :)

@jason-simmons jason-simmons added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 8, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 8, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Mar 8, 2023

auto label is removed for flutter/engine, pr: 40133, due to - Please get at least one approved review if you are already a member or two member reviews if you are not a member before re-applying this label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

@auto-submit
Copy link
Contributor

auto-submit bot commented Mar 8, 2023

auto label is removed for flutter/engine, pr: 40133, due to Validations Fail.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants