Skip to content
This repository was archived by the owner on Nov 20, 2024. It is now read-only.

avoid_private_typedef_functions support for nonfunction type aliases #2534

Closed
wants to merge 1 commit into from

Conversation

pq
Copy link
Contributor

@pq pq commented Mar 18, 2021

@google-cla google-cla bot added the cla: yes label Mar 18, 2021
@coveralls
Copy link

Coverage Status

Coverage increased (+0.001%) to 96.662% when pulling 0ce51c9 on nonfun_ta_avoid_private_typedef_functions into 4638fbe on master.

Copy link
Contributor

@bwilkerson bwilkerson left a comment

Choose a reason for hiding this comment

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

This rule seems to be very specifically about function types. I can certainly see an argument for generalizing it, but we might want to change the name and we should certainly update the documentation if we decide to go that way. I think it might be worth opening an issue to discuss the question and involve Morgan and Bob, and maybe someone from the Flutter team if it's a lint they use.

@pq
Copy link
Contributor Author

pq commented Mar 18, 2021

That sounds reasonable to me. Closing in favor of dart-lang/sdk#58354.

@pq pq closed this Mar 18, 2021
@pq pq deleted the nonfun_ta_avoid_private_typedef_functions branch July 28, 2021 17:49
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.

3 participants