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

remove composer.lock from gitignore #244

Merged
merged 1 commit into from Aug 25, 2018
Merged

remove composer.lock from gitignore #244

merged 1 commit into from Aug 25, 2018

Conversation

zaherg
Copy link

@zaherg zaherg commented Aug 6, 2018

based on getcomposer.org recommendation at https://getcomposer.org/doc/01-basic-usage.md#commit-your-composer-lock-file-to-version-control it is important to have composer.lock within git.

Committing this file to VC is important because it will cause anyone who sets up the project to use the exact same versions of the dependencies that you are using. Your CI server, production machines, other developers in your team, everything and everyone runs on the same dependencies, which mitigates the potential for bugs affecting only some parts of the deployments. Even if you develop alone, in six months when reinstalling the project you can feel confident the dependencies installed are still working even if your dependencies released many new versions since then. (See note below about using the update command.)

I know that there are some debate whether we should have this within frameworks or not, but if am not mistaken Jigsaw is an application not a framework.

More info about why we should have composer.lock within git was mentioned here https://www.engineyard.com/blog/composer-its-all-about-the-lock-file

based on getcomposer.org recommendation at https://getcomposer.org/doc/01-basic-usage.md#commit-your-composer-lock-file-to-version-control it is important to have `composer.lock` within git
@pedzed
Copy link

pedzed commented Aug 23, 2018

if am not mistaken Jigsaw is an application not a framework.

Technically, it is a framework; a set of tools pushing you to write code in a certain way. An application would mean that it is ready for end-users to use.

I would agree, it is good practice to keep the Lock file in.

@zaherg
Copy link
Author

zaherg commented Aug 24, 2018

We will always have a debate weather it is an application or a framework, as for example I cant say that Laravel Mix is a framework, its a tool, the same thing with Jigsaw in my opinion, it is just a set of tool that any user can use it, remember each one of us can define a different specification for the word end-users.

What can help us is to define who is the end-user for Jigsaw, a designer, a programmer or anyone with few knowledge of markup?

@damiani damiani merged commit cfbd685 into tighten:master Aug 25, 2018
@pedzed
Copy link

pedzed commented Aug 25, 2018

What can help us is to define who is the end-user for Jigsaw, a designer, a programmer or anyone with few knowledge of markup?

The end-user is the user at the end? I.e. the one who visits and uses the website/web application. The programmer or designer just writes it for the end-user.

I haven't ever thought that there would be a different interpretation to it, but sure. Personally I find it odd, but if you want to define it differently, that's fine.

@zaherg
Copy link
Author

zaherg commented Aug 25, 2018 via email

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.

None yet

3 participants