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.

Sign up for GitHub

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

Jump to bottom

Remove path from pubspec.yaml, it isn't valid #456

Merged
merged 1 commit into from
Aug 5, 2021
Merged

Remove path from pubspec.yaml, it isn't valid #456

merged 1 commit into from
Aug 5, 2021

Conversation

jkasten2
Copy link
Member

@jkasten2 jkasten2 commented Aug 5, 2021 •
edited
Loading

  • Path results in pub not being able to find the onesignal_flutter package.
  • Fixes the following error
    • pub get failed (66; Because onesignal_example depends on onesignal_flutter from path which doesn't exist (could not find package onesignal_flutter at "^3.1.0"), version solving failed.)
  • Tested with Flutter 2.2.3

This change is Reviewable

@jkasten2
rm path from pubspec.yaml, it isn't valid
8ab57ea 
* Path results in pub not being able to find the onesignal_flutter package.
* Fixes the following error
   - pub get failed (66; Because onesignal_example depends on onesignal_flutter from path which doesn't exist (could not find package onesignal_flutter at "^3.1.0"), version solving failed.)
* Tested with Flutter 2.2.3
@jkasten2 jkasten2 merged commit 1cc68aa into main Aug 5, 2021
@jkasten2 jkasten2 deleted the fix/example_pubspec_rm_path branch August 5, 2021 18:22
@Jeasmine Jeasmine mentioned this pull request Aug 5, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jeasmine Jeasmine Jeasmine approved these changes

@emawby emawby Awaiting requested review from emawby

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jkasten2 @Jeasmine