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

.freeze renamed to .lock? #63

Closed
mahmoud opened this issue Jan 31, 2017 · 11 comments
Closed

.freeze renamed to .lock? #63

mahmoud opened this issue Jan 31, 2017 · 11 comments

Comments

@mahmoud
Copy link

mahmoud commented Jan 31, 2017

I recall a long discussion on #43 and elsewhere, where a decent consensus was reached to change to freeze files, so as to avoid confusion with file locking. I may be confused, but this seems to have been unilaterally reverted? I can't find any further discussion.

@sethmlarson
Copy link

sethmlarson commented Jan 31, 2017

I think the reason we stuck with .lock is because it's kind of a standard for packaging (Cargo.lock etc)

@jacebrowning
Copy link

This probably isn't the place to discuss it, but I would really like to see:

pip.toml and pipfreeze.json

The filenames tells you one is the input for pip and the other is a "frozen" set of requirements. The extensions tell you the format of the file and automatically trigger syntax highlighting in editors.

@sethmlarson
Copy link

@jacebrowning Most IDEs can be configured to see a file-name and automatically interpret the structure properly. Also some are smart enough to recognize what the file format is without any name hinting at all. I can confirm that PyCharm and friends do this.

@mahmoud
Copy link
Author

mahmoud commented Jan 31, 2017

@SethMichaelLarson You say "etc." but in my years of experience with packages, I'm not familiar with another, whereas I'm quite used to seeing ".lock" used for lockfiles. I'm also concerned that this sudden change in direction occurred without a tracking issue or announcement.

@sethmlarson
Copy link

sethmlarson commented Jan 31, 2017

@mahmoud Yarn.lock? Golang uses the name lock.json for dep? Using 'lock' isn't exactly unheard of.

@mahmoud
Copy link
Author

mahmoud commented Jan 31, 2017

Considering the audience is primarily Python programmers, you may want to consider your audience. People have been pip freeze-ing long before any of these managers existed.

@sethmlarson
Copy link

But pip freeze is already sub-par because Pipfiles are so much more than module name and version. It may as well be pip list.

I see your point and I actually agree that .lock may be undesirable (for the reasons I already listed in the linked issue) but I don't think it's that big of a deal to use it. That's just my opinion. :)

@mahmoud
Copy link
Author

mahmoud commented Jan 31, 2017

Cool, thanks for closing this @kennethreitz, glad you could chime in. ☮️

@kennethreitz
Copy link
Contributor

literally every community calls it locking, so it makes sense to call it a lockfile. pipenv is already using this, and it works well.

@kennethreitz
Copy link
Contributor

anything's up to change as the project evolves though, no need to discuss it here.

@sethmlarson
Copy link

@kennethreitz Remember that pipenv and pipfile are separate projects though. Pipenv depending on things for this (still somewhat in design stage?) project shouldn't really be an argument for anything relating to pipfile.

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

No branches or pull requests

4 participants