Skip to content
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

is:wasm-ready tag #1336

Merged
merged 13 commits into from Feb 9, 2024
Merged

is:wasm-ready tag #1336

merged 13 commits into from Feb 9, 2024

Conversation

sigurdm
Copy link
Contributor

@sigurdm sigurdm commented Feb 8, 2024

Package will get the is:wasm-ready tag if it uses only the set of libraries

Only adds the is:wasm-ready tag, does not count it in scoring or mention in the report.

towards #1324

@sigurdm sigurdm requested review from isoos and jonasfj February 8, 2024 12:43
lib/src/tag/_specs.dart Outdated Show resolved Hide resolved
@sigurdm sigurdm changed the title platform:wasm tag is:wasm-ready tag Feb 8, 2024
lib/src/tag/_specs.dart Outdated Show resolved Hide resolved
lib/src/tag/pana_tags.dart Outdated Show resolved Hide resolved
@@ -20,11 +20,11 @@ Detected license: `MIT`.

## 20/20 Platform support

### [*] 20/20 points: Supports 4 of 6 possible platforms (**iOS**, **Android**, **Web**, Windows, **macOS**, Linux)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you need to rebase the PR on HEAD.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah thanks! - I was getting very confused

kevmoo
kevmoo previously requested changes Feb 8, 2024
lib/src/tag/_specs.dart Show resolved Hide resolved
@sigurdm sigurdm requested a review from kevmoo February 9, 2024 12:33
@sigurdm sigurdm dismissed kevmoo’s stale review February 9, 2024 12:34

Landing now - we can always change stuff later

@sigurdm sigurdm merged commit 65458f0 into dart-lang:master Feb 9, 2024
5 checks passed
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.

None yet

4 participants