-
Notifications
You must be signed in to change notification settings - Fork 146
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
Regarding Flutter + Flutter Web packages not being recognized on pub.dev #3277
Comments
@luanpotter: thanks for reporting this, seems like an edge case in Flutter web + dartanalyzer, we'll need to dig into it what's going on.
Could you please link where this is recommended? |
Hi @isoos , thanks for looking into this! So I'm in contact with Mariam from Google. I got some instructions and I was following this article. Mostly it suggests creating a separated package, but since I'm the owner of the original audioplayers, I thought I might do it on the same (the same way all other platforms reside on the same plugin, different implementation folders). The article directs us to add a |
Maybe the author @hterkelsen can comment on this? Or some another official Flutter channel: https://flutter.dev/community I am not aware what the official layout recommendation on this is, and I couldn't find example repositories with similar layouts. If there is anything we shall change in the analysis or suggestions, I'm happy to help with that, but I cannot recommend on how Flutter (esp. the new web platform) should be used. |
Unfortunately, the only web-specific As for layout: you could create an |
@hterkelsen, thanks for your input.
I'm not sure I follow, I am using About the layout, I understand it could be technical difficult, and I'm certainly ok with keeping |
@luanpotter: Could you please run the following commands in your package directory and copy their output here:
I have a fear that we are getting another edge case where the Flutter-included SDK slightly differs from the APIs provided by Flutter SDK. A prior bug on that was this: flutter/flutter#23664 |
There you have it:
Which seems to indicate that the |
@luanpotter: This looks looks like a Flutter-internal bug: the Flutter analyzer should recognize the missed libraries. Could you please open a bug in https://github.com/flutter/flutter/issues, copy the details over there, and also reference the bug here? Thank you! (I'll have proper time to do this on Monday if you can't do it until then.) |
@luanpotter: It looks like |
It's not clear this is a bug in pub.dev |
I'm hitting the same error. I am trying to add Web support to https://github.com/chipweinberger/flutter_pcm_sound I don't want to use a federated structure if I don't have to. Can someone upload a example repo with 'dart:web_audio' without a federated repo? Is this even possible? |
I just uploaded a new version of the package audioplayers including the flutter web files.
However, pub does not like the web files:
See: https://pub.dev/packages/audioplayers#-analysis-tab-
I also find it weird that I have to add the
audioplayer_web.dart
file inside thelib
folder. All other platforms have dedicated folders outside the lib folder. This file is the implementation for web, should be placed on aweb
folder outside thelib
folder, just likeios
,android
andmacos
. However when I do that, I cannot properly run the example app on web.Can you give me some directions on how to proceed? Having such a low score on pub is not desirable, I'm sure there is a way to configure everything properly.
The text was updated successfully, but these errors were encountered: