Visitar URL original
`sort_pub_dependencies` false positive · Issue #58241 · dart-lang/sdk · GitHub
Skip to content

sort_pub_dependencies false positive #58241

Closed
dart-archive/linter
#2283

Description

@pq

Repro:

name: animations
description: Fancy pre-built animations that can easily be integrated into any Flutter application.
version: 1.1.1
homepage: https://github.com/flutter/packages/tree/master/packages/animations

environment:
  sdk: ">=2.2.2 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

dev_dependencies:
  flutter_test:
    sdk: flutter
  vector_math: ^2.0.8

https://github.com/flutter/packages/blob/837b1a321d3f00034556ef5b385abb93c0843715/packages/animations/pubspec.yaml

See failure: https://cirrus-ci.com/task/5329170379046912

/fyi @dnfield

Activity

  1. added
    type-bugIncorrect behavior (everything from a crash to more subtle misbehavior)
    linter-false-positiveIssues related to lint rules that report a problem when it isn't a problem.
    on Oct 8, 2020
  2. self-assigned this
    on Oct 8, 2020
  3. added a commit that references this issue on Oct 8, 2020
  4. pq commented on Oct 8, 2020

    @pq
    MemberAuthor

    Hmmmm... I'm not getting this to repro locally. Looking at the reported failure, I wonder if this wasn't actually a mis-report?

    The test failure was flagging example/pubspec.yaml

    image

    which is in fact not sorted:

    https://github.com/flutter/packages/blob/837b1a321d3f00034556ef5b385abb93c0843715/packages/animations/example/pubspec.yaml

    Whereas we were looking at the root pubspec above (and that seems ok).

    🤔

    /cc @dnfield

  5. dnfield commented on Oct 8, 2020

    @dnfield
    Contributor

    Ah! you are right.

    I will file a fix for the animations package.

  6. dnfield commented on Oct 8, 2020

    @dnfield
    Contributor
  7. pq commented on Oct 8, 2020

    @pq
    MemberAuthor

    Sweet!

  8. pq commented on Oct 8, 2020

    @pq
    MemberAuthor

    When I un-revert (and for future reference), is there an easy way for me to test pro-actively if an engine roll would be blocked by a given SDK commit?

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

Metadata

Metadata

Assignees

Labels

customer-flutterdevexp-linterIssues with the analyzer's support for the linter packagelegacy-area-analyzerUse area-devexp instead.linter-false-positiveIssues related to lint rules that report a problem when it isn't a problem.type-bugIncorrect behavior (everything from a crash to more subtle misbehavior)

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions