Skip to content

Support for downloading unsigned dev builds. #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

sigmundch
Copy link
Member

Core packages, like webdev/dwds, need to run tests again dev candidate
releases that will be rolled into flutter.

This change allows users of the CI to specify whether they want to download
signed or unsigned builds. The former is what we properly call a release,
the latter is a build that is only available under the raw folder in our
cloud storage.

There are two other options here:

  • default all dev builds to use unsigned builds. I'm not sure if this
    will break on non-linux systems, so we could do it only for linux for
    example.
  • define a new name dev_unsigned and expose it as if it existed as a
    channel. This would be desugared into a dev/raw request, similar to
    how the main name is desugared here.

@athomas - what's a good way to test things out? Should I try to update one of our repos to use my fork and see if that works out? Also, should I include the release notes here or should that be done separately?

Fixes #29

/cc @annagrin

Core packages, like `webdev/dwds`, need to run tests again dev candidate
releases that will be rolled into flutter.

This change allows users of the CI to specify whether they want to download
signed or unsigned builds. The former is what we properly call a release,
the latter is a build that is only available under the `raw` folder in our
cloud storage.

There are two other options here:
* default all dev builds to use unsigned builds. I'm not sure if this
  will break on non-linux systems, so we could do it only for linux for
  example.
* define a new name `dev_unsigned` and expose it as if it existed as a
  channel.  This would be desugared into a dev/raw request, similar to
  how the `main` name is desugared here.

Fixes dart-lang#29
@sigmundch sigmundch requested a review from athomas July 8, 2021 21:31
@sigmundch
Copy link
Member Author

closing, @annagrin sent a similar PR at the same time see #36

@sigmundch sigmundch closed this Jul 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add support for dev_unpublished
1 participant