Skip to content

refactor(localize): remove deprecated canParse method from TranslationParsers#47275

Closed
alan-agius4 wants to merge 1 commit intoangular:mainfrom
alan-agius4:localize-can-parse
Closed

refactor(localize): remove deprecated canParse method from TranslationParsers#47275
alan-agius4 wants to merge 1 commit intoangular:mainfrom
alan-agius4:localize-can-parse

Conversation

@alan-agius4
Copy link
Contributor

@alan-agius4 alan-agius4 commented Aug 26, 2022

This change removed the deprecated canParse method from all the TranslationParsers.

BREAKING CHANGE:

  • canParse method has been removed from all translation parsers in @angular/localize/tools. analyze should be used instead.
  • the hint parameter in the parse methods is now mandatory.

@alan-agius4 alan-agius4 added the target: major This PR is targeted for the next major release label Aug 26, 2022
@pullapprove pullapprove bot requested a review from dgp1130 August 26, 2022 13:08
@alan-agius4 alan-agius4 changed the title refactor(@angular/localize): remove deprecated canParse method from TranslationParsers refactor(localize): remove deprecated canParse method from TranslationParsers Aug 26, 2022
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Aug 26, 2022
@alan-agius4 alan-agius4 force-pushed the localize-can-parse branch 6 times, most recently from 7b87c1c to 44cc200 Compare August 26, 2022 13:58
@alan-agius4 alan-agius4 removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Aug 26, 2022
@alan-agius4 alan-agius4 force-pushed the localize-can-parse branch 2 times, most recently from 514092b to a5a2a07 Compare August 26, 2022 14:00
@alan-agius4 alan-agius4 marked this pull request as draft August 26, 2022 14:14
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Aug 26, 2022
@alan-agius4 alan-agius4 marked this pull request as ready for review August 26, 2022 14:52
Copy link
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api

Copy link
Contributor

@AndrewKushnir AndrewKushnir left a comment

Choose a reason for hiding this comment

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

Reviewed-for: public-api

…ionParsers

This change removed the deprecated `canParse` method from all the TranslationParsers.

BREAKING CHANGE:

- `canParse` method has been removed from all translation parsers in `@angular/localize/tools`. `analyze` should be used instead.
-  the `hint` parameter in the`parse` methods is now mandatory.
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Aug 30, 2022
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit d36fd3d.

@alan-agius4 alan-agius4 deleted the localize-can-parse branch September 6, 2022 16:57
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Oct 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants