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

New package upgrade #319

Closed
wants to merge 2 commits into from
Closed

Conversation

JEuler
Copy link
Contributor

@JEuler JEuler commented Jun 11, 2020

No description provided.

@ueman
Copy link

ueman commented Jun 21, 2020

As per https://dart.dev/tools/pub/glossary#lockfile and https://dart.dev/guides/libraries/private-files the pubspec.lock file should not be put into version control because libraries should support a range of versions instead of a fixed one. So rather than changing the pubspec.lock it should be deleted and ignored.

@JEuler
Copy link
Contributor Author

JEuler commented Jun 21, 2020

As I understand, we should have lock file only for example app, correct?

@JEuler
Copy link
Contributor Author

JEuler commented Jun 21, 2020

Pushed the changes

@ryan-berger
Copy link
Collaborator

@JEuler Before I review this, please fix the merge conflicts

@JEuler JEuler force-pushed the packages-upgrade branch from fe3c25a to c55186a Compare June 28, 2020 10:00
@JEuler
Copy link
Contributor Author

JEuler commented Jun 28, 2020

Done, rebased the branch

Copy link
Collaborator

@erickok erickok left a comment

Choose a reason for hiding this comment

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

These are outdated already (not the PR creators fault of course) so this PR can be closed.

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.

4 participants