Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Conversation

dobooth
Copy link
Contributor

@dobooth dobooth commented May 20, 2020

Purpose of this pull request

This pull request (PR) updates v2.4 to use the latest supported PHP version.

whatsnew
Updated the supported versions of PHP to 7.3 and 7.4.

@dobooth dobooth self-assigned this May 20, 2020
@dobooth dobooth added 2.3.4 Magento 2.3.4 changes Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies and removed Progress: review labels May 20, 2020
@keharper
Copy link
Contributor

I see several v2.3 files in this PR. Is that intentional?

@dobooth dobooth requested a review from jeff-matthews May 21, 2020 16:27
Copy link
Contributor

@jeff-matthews jeff-matthews left a comment

Choose a reason for hiding this comment

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

I was unable to leave comments in the previously symlinked files due to the notorious "Line must be part of the diff" known issue.

  • src/guides/v2.4/install-gde/prereq/php-settings.md
    • line 14: lliquid script should point to 2.4 composer lock file.

@dobooth dobooth requested a review from jeff-matthews May 26, 2020 15:51
@jeff-matthews jeff-matthews added Major Update Significant original updates to existing content 2.4.0 and removed Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies 2.3.4 Magento 2.3.4 changes labels May 27, 2020
@dobooth
Copy link
Contributor Author

dobooth commented May 27, 2020

running tests

@dobooth dobooth merged commit 0bb9247 into 2.4.0-develop May 27, 2020
@ghost
Copy link

ghost commented May 27, 2020

Hi @dobooth, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@dobooth dobooth deleted the db_php74_2.4develop branch May 27, 2020 19:34
@hunt0r
Copy link

hunt0r commented Oct 6, 2020

I feel silly posting this here, so please delete if it's inappropriate.
@jeff-matthews your post above references 'the notorious "Line must be part of the diff" known issue'. I have encountered that issue, but unable to find it with Google/Github support searching. My Google-searching led me here 😆 Would you mind sharing a link about it?

@jeff-matthews
Copy link
Contributor

Hi @hunt0r! I wish I could share a link, but I was actually referring to this being "notorious" for those of us working on this particular project. I think everyone on the devdocs-admins team has encountered it at some point.

All we really know is that it has something to do with files that were previously symlinked in our project. For example, after deleting the symlink for src/guides/v2.4/install-gde/prereq/php-settings.md, creating a new/"real" file, and adding/pushing, the GitHub interface throws that error if you try to leave inline comments during review.

We haven't been able to track down any official root cause. So if you find something, please come back and let us know!

@hunt0r
Copy link

hunt0r commented Oct 7, 2020

That is exactly my case. Unfortunately, it is on a private repo with proprietary code so I can't provide an example.
We had one "file" which was merely a symlink to another.
I deleted the symlink and then copied another file into the same place. Git seemed to handle this fine.
I pushed to github, and then went to write a comment on one line of the file. But got that error, and could not post my comment.

Thanks for the response. Since I can't provide an example, I am hesitant to open a Github support ticket. But if you are willing to open one, please let me know if I can help in some way.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.4.0 Major Update Significant original updates to existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants