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
Change instances of oop5 to oop #255
base: master
Are you sure you want to change the base?
Conversation
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.
LGTM if this builds with the doc-base changes, as this is basically just a search and replace.
<!-- <link linkend="language.oop5.basic.class.class">::class</link> --> | ||
<!-- <link linkend="language.oop.basic.class.class">::class</link> --> |
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 should just be removed, that's a leftover from one of my copy/pastes
Something that occurred to me, since some of these changes will be unnecessary when the other PHP 5 removal PRs are merged, some of the files I've changed in this PR (e.g. migration guides) will be removed. Should I remove those changes from this PR? |
I believe that this PR is mergeable as long as it's rebased to take care of conflicts and recently removed files. |
Translators will hate us for merging this. |
Can't we reproduce the same on the translation repos? It's the kind of change that doesn't require actually speaking the language. |
The issue is tracking the revision changes. As an automatic "fix" which consists of moving files when the translation of said files is not up to date is basically useless, so one would need to ensure all translations have all files in oop5/ up to date. And then there is the story about changing the IDs, that can be done but one cannot update the revision/commit hash for the same reasons as above |
Pinging @tiffany-taylor in case this fell through the cracks |
RL is currently taking most of my attention. I will try to come back to this in a few weeks. |
|
With regards to #255 (comment), if a translation doc repo is out of date, what is the best course of action for me to mitigate the translation maintainers workload with this change? |
This PR seems ambitious and has many merge conflicts after 2 years. How valuable is this change compared to the effort and possible disruption for translators? I feel like something so big requires a strategy before we make the change. What are your thoughts? |
I still think this is a valuable change, but as with every QA change of that sort, leaving it to be done around spring is usually the best as hopefully translations will be up to date in regards to the oop sections as not that much changes should happen now that most of the updates for 8.2 should have landed. But will need some coordinations with the translations (RU: @saundefined JP: @mumumu FR: myself, and the other languages I don't really know who's in charge) |
I'll set myself a reminder for March 15th :) |
Past Me was way too optimistic. I'm not able to push this like I had hoped, but I will try to update the branch to a more recent commit within the next month. |
@tiffany-taylor Is this still something that you want to try to complete? It's been sitting here for some time. Perhaps the sunk cost fallacy is at play here :) |
It's not a very difficult change to enact, it's just taxing on translations. And trying to find the correct moment for this is always a bit tricky |
This changes instances of oop5 to oop.
This includes text within XML files, renaming the directory language/oop5/ to language/oop/, and renaming oop5.xml to oop.xml.
This PR is dependent on a change to the doc-base repo which also updates oop5 to oop. I will make a separate PR for that.