Skip to content

Meta-issue: Update all code to use optional new/const #18604

Closed

Description

@mit-mit

Tracking issue for updating all our Flutter example code to use optional new/const (now that we are unblocked by the needed lint dart-lang/sdk#32602 being available):

  • Flutter.io homepage
  • flutter/flutter/examples
  • flutter/plugins/examples
  • flutter/packages/examples

Activity

  1. a14n commented on Jun 19, 2018

    @a14n
    Contributor

    Actually it's still blocked by dart-lang/sdk#57706 and dart-lang/sdk#57712

  2. mit-mit commented on Jun 20, 2018

    @mit-mit
    MemberAuthor

    Thanks, I reactivated dart-lang/sdk#32596

  3. pq commented on Jun 26, 2018

    @pq
    Contributor

    @a14n : once the new linter makes it's way into a vended SDK will we be good to make these changes?

  4. a14n commented on Jun 27, 2018

    @a14n
    Contributor

    I tried to remove unnecessary new/const from flutter codebase and faced a new issue in analyzer dart-lang/sdk#33647

  5. a14n commented on Jul 16, 2018

    @a14n
    Contributor

    Optional const landed in #19342

  6. pq commented on Jul 16, 2018

    @pq
    Contributor

    @a14n: this is fantastic. Remind me the plan for unnecessary_new?

  7. mit-mit commented on Aug 6, 2018

    @mit-mit
    MemberAuthor

    @a14n do we have the no new lint?

  8. a14n commented on Aug 6, 2018

    @a14n
    Contributor

    unnecessary_const landed with #20103 and flutter-team-archive/plugins#682.
    unnecessary_new is waiting for reviews with #20138 and flutter-team-archive/plugins#701.

  9. a14n commented on Oct 5, 2018

    @a14n
    Contributor

    unnecessary new/const have been removed from flutter/flutter, flutter/plugins and soon in flutter/packages.
    On flutter.io the only remaining place with new/const seems to be in images like on home page at the Fast development section.

  10. mit-mit commented on Oct 8, 2018

    @mit-mit
    MemberAuthor

    Excellent! I'll make sure with @maguinis that we get all images updated!

  11. mit-mit commented on Oct 15, 2018

    @mit-mit
    MemberAuthor

    Marking flutter/packages done as flutter/packages#4 was merged

  12. mit-mit commented on Oct 15, 2018

    @mit-mit
    MemberAuthor

    Assigning to self as only item left is updating images on flutter.io

  13. self-assigned this
    on Oct 15, 2018
  14. a14n commented on Oct 15, 2018

    @a14n
    Contributor

    Actually there's still flutter-team-archive/plugins#818

  15. mit-mit commented on Nov 14, 2018

    @mit-mit
    MemberAuthor

    This should be done now

  16. github-actions commented on Sep 1, 2021

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  17. locked as resolved and limited conversation to collaborators on Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

c: contributor-productivityTeam-specific productivity, code health, technical debt.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions