Skip to content

Conversation

fballiano
Copy link

No description provided.

@addison74
Copy link
Contributor

It was absolutely necessary to keep a list of the differences between the main and next branches. You have certainly kept records and you can check if there are others. Those who use OM must be prepared for changes in the future, I think that very few use the next branch in testing. This list will help them.

@fballiano
Copy link
Author

@fballiano fballiano requested a review from addison74 July 17, 2023 20:36
addison74
addison74 previously approved these changes Jul 17, 2023
@fballiano
Copy link
Author

that point is now the only one in the entire readme that has a ".", it doesn't look good to me, that's why I preferred one sentence. I don't think we've to say to backup, still.

@addison74
Copy link
Contributor

Often when things don't go well people tend to blame others. By mentioning to backup the files is the way to avoid further discussions.

@fballiano
Copy link
Author

it's written everywhere to backup when updating. anyway. I still think it looks off to have only one dot in the lists, but ok

@addison74
Copy link
Contributor

You can remove the last dot (after the "script" word) to keep the same format. All the others are fine.

@addison74
Copy link
Contributor

Offtopic: #3146 - This merged PR should be in the main branch. It is a very useful feature. Moving a website to the next branch will involve effort. Jquery is 3.7 instead of 1.12, used in most existing extensions. this in addition to bringing extensions to PHP 8.2. Therefore, until the transition is made, certain PRs should be merged in the main branch, as is the one mentioned by me.

@fballiano
Copy link
Author

this one is already on the main branch, #3146 is a breaking change and since the famous RFC we've to put breaking changes in the next branch :-)

@fballiano fballiano merged commit f69f6fc into OpenMage:main Aug 18, 2023
@fballiano fballiano deleted the readmenext branch August 18, 2023 14:17
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.

4 participants