Skip to content

pkg:pigeon and pkg:flutter_markdown fail publishable check #82368

Closed

Description

@goderbauer

pkg:flutter_markdown

* 1 checked in file are ignored by a `.gitignore`.
  Previous versions of Pub would include those in the published package.
  Consider adjusting your `.gitignore` files to not ignore those files, and if you do not wish to
  publish these files use `.pubignore`. See also dart.dev/go/pubignore
  Files that are checked in while gitignored:
  flutter_markdown.iml

pkg:pigeon

* 4 checked in files are ignored by a `.gitignore`.
  Previous versions of Pub would include those in the published package.
  Consider adjusting your `.gitignore` files to not ignore those files, and if you do not wish to
  publish these files use `.pubignore`. See also dart.dev/go/pubignore
  Files that are checked in while gitignored:
  ci/.gitignore
  e2e_tests/test_objc/ios/Flutter/AppFrameworkInfo.plist
  e2e_tests/test_objc/ios/Flutter/Debug.xcconfig
  e2e_tests/test_objc/ios/Flutter/Release.xcconfig

This is keeping the flutter/packages tree red: https://cirrus-ci.com/task/6592972780208128?logs=publishable#L1337

Activity

  1. goderbauer commented on May 12, 2021

    @goderbauer
    MemberAuthor

    @gaaclarke for pigeon
    @stuartmorgan and @domesticmouse for flutter_markdown

  2. added
    packageflutter/packages repository. See also p: labels.
    p: pigeonrelated to pigeon messaging codegen tool
    on May 12, 2021
  3. stuartmorgan-g commented on May 12, 2021

    @stuartmorgan-g
    Contributor

    Looks like a new warning. I'll do an audit of the repo's .gitignore since I'm not sure all of those should be ignored.

  4. stuartmorgan-g commented on May 12, 2021

    @stuartmorgan-g
    Contributor

    The issues are both local to the packages. flutter_markdown just needs that file removed.

    @gaaclarke why does pigeon have e2e_tests/test_objc/ios/Flutter/ in its gitignore? Are you generating the app on the fly each time it's run or something? If not, that directory shouldn't be ignored, only the generated files in it. For some reason the test app doesn't have the template-generated .gitignore that would handle things like this correctly.

  5. gaaclarke commented on May 12, 2021

    @gaaclarke
    Member

    @stuartmorgan if I recall correctly, it was done to ignore the App.framework which is regenerated. I'll clean it up.

  6. gaaclarke commented on May 12, 2021

    @gaaclarke
    Member

    I'm having a hard time verifying the results locally:

    $ ./script/tool_runner.sh publish-check
    Unhandled exception:
    Could not find an option named "run-on-changed-packages".
    
    Usage: pub global run flutter_plugin_tools <command> [arguments]
    ...

    I upgraded my flutter_plugin_tools but no luck

  7. gaaclarke commented on May 12, 2021

    @gaaclarke
    Member

    I think pub publish --dry-run without warnings should catch it (assuming I have the right version of dart).

  8. stuartmorgan-g commented on May 12, 2021

    @stuartmorgan-g
    Contributor

    I upgraded my flutter_plugin_tools but no luck

    What version did it say you were activating?

    But yes, pub publish --dry-run is pretty much all that's doing.

  9. gaaclarke commented on May 12, 2021

    @gaaclarke
    Member

    I did this dart pub global activate --source git https://github.com/flutter/plugin_tools.git hoping it would get the latest, it got 0.0.45+3.

  10. stuartmorgan-g commented on May 12, 2021

    @stuartmorgan-g
    Contributor

    That repo is discontinued; the source moved to flutter/plugins because it's much harder to maintain out-of-repo (relative to most of its use; flutter/packages is a subset, and has far fewer PRs). You want dart pub global activate flutter_plugin_tools.

  11. stuartmorgan-g commented on May 13, 2021

    @stuartmorgan-g
    Contributor

    Fixed by the PRs above.

  12. github-actions commented on Aug 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.

  13. locked as resolved and limited conversation to collaborators on Aug 1, 2021
  14. added
    P0Critical issues such as a build break or regression
    and removed on Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P0Critical issues such as a build break or regressionp: flutter_markdownflutter/packages flutter_markdownp: pigeonrelated to pigeon messaging codegen toolpackageflutter/packages repository. See also p: labels.team-ecosystemOwned by Ecosystem team

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions