Skip to content
This repository has been archived by the owner on Jan 6, 2025. It is now read-only.

Add package:js to dependencies #71

Merged
merged 7 commits into from
Dec 14, 2022

Conversation

elliette
Copy link
Contributor

@elliette elliette commented Dec 14, 2022

Fixes the following error when I went to publish package:sse:

Publishing sse 4.1.2 to https://pub.dev:
....
Package validation found the following error:
* line 9, column 1 of lib/client/sse_client.dart: This package does not have js in the `dependencies` section of `pubspec.yaml`.
    ╷
  9 │ import 'package:js/js.dart';
    │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Also updates the min Dart SDK to 2.16.0 for version compatibility with package:js

@elliette elliette requested a review from grouma December 14, 2022 00:02
@elliette elliette requested a review from natebosch December 14, 2022 17:42
pubspec.yaml Outdated
@@ -18,6 +18,7 @@ dependencies:
stream_channel: '>=1.6.8 <3.0.0'

dev_dependencies:
js: ^0.6.4
Copy link
Contributor

Choose a reason for hiding this comment

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

This is used under lib/, so it should be in dependencies instead of dev_dependencies

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, thank you!

@elliette elliette changed the title Add package:js to dev dependencies Add package:js to dependencies Dec 14, 2022
@elliette elliette merged commit cfa93b1 into dart-archive:master Dec 14, 2022
mosuem pushed a commit to dart-lang/tools that referenced this pull request Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants