- Notifications
You must be signed in to change notification settings - Fork 929
[#266] Set minimum-stability to stable and added docs. #667
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
Conversation
d416363 to 425a118 Compare composer.json Outdated
| "drupal/core-composer-scaffold": "^11@alpha", | ||
| "drupal/core-recommended": "^11@alpha", | ||
| "drush/drush": "^13@beta", | ||
| "chi-teck/drupal-code-generator": "4.x-dev#e61b917 as 3.5.0", |
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.
This package was updated to Symfony 7, but there is no release.
Considering that the whole 11.x branch is "early preview", I think this should be okay to have like this.
Once Drupal 11 is in beta and all these packages have their releases, we can clean it up.
425a118 to c0ac5e8 Compare composer.json Outdated
| } | ||
| ], | ||
| "require": { | ||
| "chi-teck/drupal-code-generator": "4.x-dev#e61b917 as 3.5.0", |
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.
This package was updated to Symfony 7, but there is no release.
Considering that the whole 11.x branch is "early preview", I think this should be okay to have like this.
Once Drupal 11 is in beta and all these packages have their releases, we can clean it up.
I've created an issue - maybe we can wait for a couple of days before merging this one.
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 don't think you need to add as 3.5.0 unless another package explicitly needs that version. If you are using composer require ... it won't work because it checks stability. But if you just add it manually to the composer.json it works fine. This is a feature afaik.
To test, I updated it locally to be this and ran composer update.
"chi-teck/drupal-code-generator": "4.x-dev", I don't know if pinning it to a commit hash is good... if the package maintainers happen to rewrite history on the dev branch then composer will stop building?
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.
it worked for you because you probably did not use the dev version of core.
Drush 13.x requires 4.x-dev for Symfony 7 compatibility https://github.com/drush-ops/drush/blob/13.x/composer.json#L36C3-L36C57
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.
chi-teck/drupal-code-generator is now released. waiting for Drush to start using it at alpha+ version level.
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.
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.
it worked for you because you probably did not use the dev version of core.
Sorry I isn't sinking in why this project needs to use the dev version of core.
| } | ||
| ], | ||
| "minimum-stability": "dev", | ||
| "minimum-stability": "alpha", |
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 like this change.
cc60531 to f5a4641 Compare minimum-stability to true and added docs.minimum-stability to stable and added docs. 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.
Nice!! Very clear documentation, I like. I'm also in favor of waiting for new code generator release instead of getting the alias committed. But when there's a follow-up issue so we don't forget about it I'm also okay with it in the meanwhile.
f5a4641 to fca7327 Compare | @leymannx |
fca7327 to 2b64fbc Compare
Closes #266