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

Fix dependency on discontinued js package #3861

Open
kirkmorrow opened this issue Feb 14, 2025 · 12 comments
Open

Fix dependency on discontinued js package #3861

kirkmorrow opened this issue Feb 14, 2025 · 12 comments
Assignees
Labels
type-enhancement A request for a change that isn't a bug

Comments

@kirkmorrow
Copy link

The package js, which build_runner has a dependency on, was marked discontinued with release 0.7.2 on Feb. 12, 2025. The recommendation is to use js_interop instead moving forward per https://pub.dev/packages/js.

@kirkmorrow kirkmorrow added the type-enhancement A request for a change that isn't a bug label Feb 14, 2025
@jakemac53
Copy link
Contributor

This should have been completed in #3793

@lishaduck
Copy link

lishaduck commented Feb 14, 2025

@jakemac53, that removed actual usage of js, but it's still listed as a dependency:

js: '>=0.6.3 <0.8.0'


And it looks like build_web_compilers still depends on it as well.

@jakemac53 jakemac53 reopened this Feb 14, 2025
@jakemac53
Copy link
Contributor

Weird, I didn't see any more diagnostics, but yeah any remaining usage should be removed

@jakemac53
Copy link
Contributor

Oh you are saying it's an unused dependency that just didn't get cleaned up

@lishaduck
Copy link

Oh you are saying it's an unused dependency that just didn't get cleaned up

In build_runner, yes, in build_web_compilers, there's an actual use (but there's an ignore_deprecated_member_use, so that's probably why you didn't see diagnostics)

@lishaduck
Copy link

lishaduck commented Feb 14, 2025

Here's build_web_compilers:

// ignore: deprecated_member_use
import 'package:js/js.dart';

@jakemac53
Copy link
Contributor

ohhh, we just ignored it 🤕

@davidmorgan davidmorgan self-assigned this Feb 15, 2025
@davidmorgan
Copy link
Contributor

Yes, I added the ignore as it broke CI :)

I guess it's okay if I get to this in a week or two, nothing's on fire?

@lishaduck
Copy link

I don't depend on build_web_compilers, but can we at least remove the unused dependency on js in build_runner on Monday?

(It's not breaking me, it's just that it's a tiny, separate patch that I'm sure others will hit pretty soon. Enjoy your weekend!)

@davidmorgan
Copy link
Contributor

They'll just see a warning, I think? Or is there actual breakage?

@lishaduck
Copy link

They'll just see a warning, I think? Or is there actual breakage?

Just a warning, but again, it's a one-line fix and I'm certain some people will read "deprecated" and freak out (see: npm ecosystem).

@davidmorgan
Copy link
Contributor

I'm in the middle of a huge refactoring, it's not straightforward to do a release right now. Hopefully this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

4 participants