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

TASK: Require all collection packages as self.version #3035

Merged

Conversation

kdambekalns
Copy link
Member

Any dependency from one package in the collection to another should always be "to the same version".

This changes makes sure this is the case by using self.version as the required version.

See #3034

Checklist

  • Code follows the PSR-2 coding style
  • Tests have been created, run and adjusted as needed
  • The PR is created against the lowest maintained branch
  • Reviewer - PR Title is brief but complete and starts with FEATURE|TASK|BUGFIX
  • Reviewer - The first section explains the change briefly for change-logs
  • Reviewer - Breaking Changes are marked wit !!! and have upgrade-instructions

@kitsunet kitsunet merged commit 5b927a2 into neos:7.3 May 11, 2023
9 of 10 checks passed
Comment on lines +17 to +28
"neos/cache": "self.version",
"neos/eel": "self.version",
"neos/error-messages": "self.version",
"neos/utility-arrays": "self.version",
"neos/utility-files": "self.version",
"neos/utility-mediatypes": "self.version",
"neos/utility-objecthandling": "self.version",
"neos/utility-opcodecache": "self.version",
"neos/utility-schema": "self.version",
"neos/utility-unicode": "self.version",
"neos/flow-log": "self.version",
"neos/http-factories": "self.version",
Copy link
Member

Choose a reason for hiding this comment

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

This change was lost with the release on 29.06: ee09e21. @kdambekalns did you expect this to be changed during releases?

Copy link
Member Author

Choose a reason for hiding this comment

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

No, didn't expect that… 😥

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, the release scripts… I did not think of those. 🤦‍♂️

@kdambekalns kdambekalns deleted the task/require-self-version-inside-collection branch September 28, 2023 17:41
kdambekalns added a commit to kdambekalns/flow-development-collection that referenced this pull request Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants