Skip to content

Prepare to release dwds 18.0.0 #1993

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

Merged
merged 7 commits into from
Feb 27, 2023
Merged

Conversation

annagrin
Copy link
Contributor

Prepare to release dwds 18.0.0

@annagrin
Copy link
Contributor Author

annagrin commented Feb 24, 2023

@kevmoo I run into a snug using publish here, so currently it is not running: dart-lang/ecosystem#75

I am planning to just manually publish dwds after this PR is approved. Do I need to perform some manual verification steps?

@@ -11,4 +11,4 @@ on:
jobs:
publish:
if: ${{ github.repository_owner == 'dart-lang' }}
uses: dart-lang/ecosystem/.github/workflows/publish.yaml@main
uses: dart-lang/ecosystem/.github/workflows/publish.yaml@dev
Copy link
Member

Choose a reason for hiding this comment

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

You'll want to retain @main here; that's referring to the repo, path, and git branch of where the script is (we don't have a dev branch or tag on that repo).

Copy link
Member

Choose a reason for hiding this comment

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

I believe what you want here (in order to publish from a beta channel sdk version) is to specify:

    with:
      sdk: beta

ala https://github.com/dart-lang/ecosystem/tree/main/pkgs/firehose#publishing-from-a-specific-version-of-the-sdk. Hope that helps :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

got it, thanks!

Copy link
Contributor Author

@annagrin annagrin Feb 25, 2023

Choose a reason for hiding this comment

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

Hmm tried that, got a warning that failed the validation:

  Validating package...
  Package validation found the following potential issue:
  * Packages with an SDK constraint on a pre-release of the Dart SDK should themselves be published as a pre-release version. If this package needs Dart version 3.0.0-188.0.dev, consider publishing the package as a pre-release instead.
    See https://dart.dev/tools/pub/publishing#publishing-prereleases For more information on pre-releases.
  
  Package has 1 warning.

Should we be failing on this warning for the lower bound SDK constraint on the package? This release is in preparation for dart 3.0 and needs to be a stable release to be rolled into flutter, which still has a pre-3.0 dart, but we don't want it to be accidentally updated for users before they switch to the right SDK.

@annagrin annagrin merged commit 4e08276 into dart-lang:master Feb 27, 2023
@annagrin annagrin deleted the release-dwds branch March 22, 2023 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants