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

Drop lock files #897

Merged
merged 3 commits into from May 27, 2022
Merged

Drop lock files #897

merged 3 commits into from May 27, 2022

Conversation

domesticmouse
Copy link
Contributor

Pre-launch Checklist

  • I read the Effective Dart: Style recently, and have followed its advice.
  • I signed the CLA.
  • I updated/added relevant documentation (doc comments with ///).
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-devrel channel on Discord.

@@ -7,3 +7,8 @@
*/**/.metadata

.dart_tool/

# Lockfiles
Podfile.lock
Copy link
Contributor

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.

Copy link
Contributor Author

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.

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:

  1. 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.
  2. The project in question is a real production app with regular, distributed development. In our case, these are mostly idle samples.

Copy link

@craiglabenz craiglabenz left a comment

Choose a reason for hiding this comment

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

LGTM

@domesticmouse domesticmouse merged commit bb73e58 into flutter:main May 27, 2022
8 checks passed
@domesticmouse domesticmouse deleted the drop-lockfiles branch May 27, 2022 21:00
@domesticmouse domesticmouse mentioned this pull request May 28, 2022
4 tasks
@derKosi
Copy link

derKosi commented Jun 1, 2022

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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants