Skip to content

Prepare to release dwds 18.0.0#1993

Merged
annagrin merged 7 commits into
dart-lang:masterfrom
annagrin:release-dwds
Feb 27, 2023
Merged

Prepare to release dwds 18.0.0#1993
annagrin merged 7 commits into
dart-lang:masterfrom
annagrin:release-dwds

Conversation

@annagrin

Copy link
Copy Markdown
Contributor

Prepare to release dwds 18.0.0

@annagrin

annagrin commented Feb 24, 2023

Copy link
Copy Markdown
Contributor Author

@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?

Comment thread .github/workflows/publish.yaml Outdated
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
Copy Markdown
Contributor

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
Copy Markdown
Contributor

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
Copy Markdown
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!

@annagrin annagrin Feb 25, 2023

Copy link
Copy Markdown
Contributor Author

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