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

Update emitter.dart #429

Merged
merged 3 commits into from
Sep 25, 2023
Merged

Update emitter.dart #429

merged 3 commits into from
Sep 25, 2023

Conversation

mosuem
Copy link
Contributor

@mosuem mosuem commented Sep 21, 2023

Add a newline after lambdas.


  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

Add a newline after lambdas.
Copy link
Contributor

@devoncarew devoncarew left a comment

Choose a reason for hiding this comment

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

lgtm

cc @natebosch

@github-actions
Copy link

Package publishing

Package Version Status Publish tag (post-merge)
package:code_builder 4.7.0 ready to publish v4.7.0

Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.

@mosuem mosuem merged commit d839123 into master Sep 25, 2023
@mosuem mosuem deleted the mosuem-patch-1 branch September 25, 2023 11:50
@natebosch
Copy link
Contributor

What was the motivation for this?

I don't specifically disagree with the change - but I'm just now noticing the impact as I'm trying to roll it in to google3. Is it worth breaking existing golden tests for this change?

@natebosch
Copy link
Contributor

Followed the backlink to dart-lang/i18n#721 (comment)

I do think the change makes sense. I'll work on the golden files.

mosuem added a commit to dart-lang/tools that referenced this pull request Oct 25, 2024
* Update emitter.dart

Add a newline after lambdas.

* Rev version

* Fix typo
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants