Skip to content

[Intl] Add missing entries to intl upgrade log #41362

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

Closed
wants to merge 1 commit into from
Closed

[Intl] Add missing entries to intl upgrade log #41362

wants to merge 1 commit into from

Conversation

malteschlueter
Copy link
Contributor

Q A
Branch? 5.3
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

This is a follow up of #41361.

@nicolas-grekas
Copy link
Member

All these classes are internal AFAIK. If confirmed, there is no need for these lines.

@malteschlueter
Copy link
Contributor Author

malteschlueter commented May 21, 2021

All these classes are internal AFAIK. If confirmed, there is no need for these lines.

Good to know, thank you^^

Only the deleted Exceptions are not @internal but they just used in the deleted so it doesn't matter or?

Intl
----------

* Deprecate `DateFormatter` namespace, use symfony/polyfill-intl-icu ^1.21 instead
Copy link
Member

Choose a reason for hiding this comment

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

These are internal classes. What should be mentioned are the polyfill classes for PHP class names

@malteschlueter
Copy link
Contributor Author

Okay, I will close this PR. Thank you @nicolas-grekas and @stof

@malteschlueter malteschlueter deleted the 6.0-intl-deprecated-upgrade-md branch May 22, 2021 13:42
@stof
Copy link
Member

stof commented May 22, 2021

There is still a need to mention that the polyfills bundled in the component are deprecated, and that you shuld install the polyfill package if you rely on them.

@stof
Copy link
Member

stof commented May 22, 2021

for the exception, the @internal tag should be added IMO. not having it was a mistake.

nicolas-grekas added a commit that referenced this pull request May 27, 2021
This PR was merged into the 6.0 branch.

Discussion
----------

[Intl] Removed deprecated code

| Q             | A
| ------------- | ---
| Branch?       | 6.0
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

This remove deprecated code from Intl.

TODO
- [x] Update UPGRADE-5.3.md and UPGRADE-6.0.md in Branch 5.3 #41362

Commits
-------

87f785b [Intl] Removed deprecated code
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