Skip to content

I18n extract default build target#27558

Merged
clydin merged 3 commits intoangular:mainfrom
clydin:i18n-extract/default-build-target
Apr 29, 2024
Merged

I18n extract default build target#27558
clydin merged 3 commits intoangular:mainfrom
clydin:i18n-extract/default-build-target

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Apr 29, 2024

The extract-i18n option schema has been updated to reflect that the
buildTarget option is not required. The option value will default
to the build target of the containing project. This removes the need
for an additional option and character count within the angular.json
file for the project.

clydin added 3 commits April 29, 2024 10:24
… required

The `extract-i18n` option schema has been updated to reflect that the
`buildTarget` option is not required. The option value will default
to the `build` target of the containing project. This removes the need
for an additional option and character count within the `angular.json`
file for the project.
…on is no longer required

The `extract-i18n` option schema has been updated to reflect that the
`buildTarget` option is not required. The option value will default
to the `build` target of the containing project. This removes the need
for an additional option and character count within the `angular.json`
file for the project. This change also applies to the deprecated
`browserTarget` option.
…et option for new applications

The `buildTarget` option for the `extract-18n` builder will automatically
default to use the `build` target of the containing project. This removes
the need for projects to manually configure the target. New projects will
now be generated without this option in the `angular.json` file.
@clydin clydin force-pushed the i18n-extract/default-build-target branch from 80789d6 to 9fb1938 Compare April 29, 2024 14:31
@clydin clydin added the target: rc This PR is targeted for the next release-candidate label Apr 29, 2024
@clydin clydin marked this pull request as ready for review April 29, 2024 14:38
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Apr 29, 2024
@clydin clydin requested a review from alan-agius4 April 29, 2024 14:38
@clydin clydin 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 Apr 29, 2024
@clydin clydin merged commit b0ec254 into angular:main Apr 29, 2024
@clydin clydin deleted the i18n-extract/default-build-target branch April 29, 2024 16:10
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 May 30, 2024
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: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants