Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Reorganize UPGRADE-1.9 file to extract steps for Symfony v5.2#12332

Merged
pamil merged 1 commit into
Sylius:1.9from
GSadee:upgrade-19-reorganize
Feb 11, 2021
Merged

Reorganize UPGRADE-1.9 file to extract steps for Symfony v5.2#12332
pamil merged 1 commit into
Sylius:1.9from
GSadee:upgrade-19-reorganize

Conversation

@GSadee
Copy link
Copy Markdown
Member

@GSadee GSadee commented Feb 11, 2021

Q A
Branch? 1.9
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

@GSadee GSadee added DX Issues and PRs aimed at improving Developer eXperience. Maintenance CI configurations, READMEs, releases, etc. labels Feb 11, 2021
@GSadee GSadee requested a review from a team as a code owner February 11, 2021 08:36
@GSadee GSadee force-pushed the upgrade-19-reorganize branch from 8b45719 to 52fd6fb Compare February 11, 2021 08:56
Comment thread UPGRADE-1.9.md Outdated
Comment thread UPGRADE-1.9.md
Comment thread UPGRADE-1.9.md Outdated
Comment thread UPGRADE-1.9.md

1. We've removed the support for Symfony's Templating component (which is removed in Symfony 5).
```bash
composer require "friendsofsymfony/oauth-server-bundle" ">2.0.0-alpha.0 ^2.0@dev"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it required when staying with Symfony 4.4?

Copy link
Copy Markdown
Member Author

@GSadee GSadee Feb 11, 2021

Choose a reason for hiding this comment

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

It looks as it is because without this requirement I have friendsofsymfony/oauth-server-bundle in 1.6.2 version and that kind of error: https://github.com/GSadee/TestUpgrade/runs/1877377582?check_suite_focus=true#step:22:10

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah, that's right :/

@GSadee GSadee force-pushed the upgrade-19-reorganize branch from 52fd6fb to 16b9d64 Compare February 11, 2021 09:59
@pamil pamil merged commit 382f2ee into Sylius:1.9 Feb 11, 2021
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Feb 11, 2021

Thank you, Grzegorz! 🎉

@GSadee GSadee deleted the upgrade-19-reorganize branch February 11, 2021 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DX Issues and PRs aimed at improving Developer eXperience. Maintenance CI configurations, READMEs, releases, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants