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
Drop lock files #897
Drop lock files #897
Conversation
| @@ -7,3 +7,8 @@ | |||
| */**/.metadata | |||
|
|
|||
| .dart_tool/ | |||
|
|
|||
| # Lockfiles | |||
| Podfile.lock | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was expecting to see pubspec.lock added here, but the other lock files were a surprise to me.
At the risk of opening up another can of worms, are there risks to taking those out as well? The Cocoapods docs are pretty adamant about always including theirs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Podfile.lock is generated by the flutter tooling at compile time, and depends on the dependency versions in pubspec.lock.
You've linked the reasoning about checking in the Cocoapod dependencies, which we don't do.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question, @RedBrogdon. I had no opinion about this before reading the docs and @domesticmouse's response, but after doing so, I think I agree with Brett. I think Cocoapods' docs are written with two big assumptions that are both untrue in our use case:
- They live at the root of the project. In our case, the equivalent is
pubspec.yaml, with all Cocoapods things being generated - as Brett said. - The project in question is a real production app with regular, distributed development. In our case, these are mostly idle samples.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Lab missing instruction https://codelabs.developers.google.com/codelabs/flutter-github-client?hl=en#5 Does not have a instructions to extend or change the interface nor channel or test dart files. Thus beginners could get frustrated. (Also I am not sure about the window to front platform specific code, but i have not tested on all platforms, maybe it is just right) |
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-devrel channel on Discord.