Skip to content

Add real mustachio annotation, and check in in-progress renderers. #2430

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 17, 2020

Conversation

srawlins
Copy link
Member

  • Add mustachio to build.yaml
  • Add a class Renderer, meant to be used as an annotation
  • Add one @Renderer annotation to templates.dart
  • Add the generated templates.renderers.dart file. While in-progress, this
    library is valid Dart, and passes static analysis. Checking it in ensures
    that it will continue to be valid Dart, and pass static analysis.
  • Remove the abstract getFields method; the generated code doesn't implement
    it yet; this was checked in prematurely.
  • Remove the templateUri field from the mock Renderer class in the tests; this
    field won't be needed; template URIs are provided at runtime.

* Add mustachio to build.yaml
* Add a class Renderer, meant to be used as an annotation
* Add one @Renderer annotation to templates.dart
* Add the generated templates.renderers.dart file. While in-progress, this
  library is valid Dart, and passes static analysis. Checking it in ensures
  that it will continue to be valid Dart, and pass static analysis.
* Remove the abstract `getFields` method; the generated code doesn't implement
  it yet; this was checked in prematurely.
* Remove the templateUri field from the mock Renderer class in the tests; this
  field won't be needed; template URIs are provided at runtime.
@google-cla google-cla bot added the cla: yes Google CLA check succeeded. label Nov 17, 2020
@srawlins srawlins requested a review from jcollins-g November 17, 2020 14:22
@srawlins srawlins merged commit 26480a3 into dart-lang:master Nov 17, 2020
@srawlins srawlins deleted the mustachio-4 branch November 17, 2020 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Google CLA check succeeded.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants