You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The reason will be displayed to describe this comment to others. Learn more.
@ihor-sviziev can I ask you for that changes? why the version should remove from composer.json?
Looks like the version in composer.json indicates which version is it.
If the version removes from composer.json so everyone doesn't know which version they installed and we can't get module updates automatically?
The reason will be displayed to describe this comment to others. Learn more.
@Viper9x,
The version inside composer.json overrides the version you actually trying to install. If it’s not defined - packagist using git branches and tags to check which version you would like to install.
So basically it’s not needed for composer
3fdfba2
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.
@ihor-sviziev can I ask you for that changes? why the version should remove from composer.json?
Looks like the version in composer.json indicates which version is it.
If the version removes from composer.json so everyone doesn't know which version they installed and we can't get module updates automatically?
3fdfba2
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.
@Viper9x,
The version inside composer.json overrides the version you actually trying to install. If it’s not defined - packagist using git branches and tags to check which version you would like to install.
So basically it’s not needed for composer
3fdfba2
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.
@ihor-sviziev thanks for your information.