Skip to content

fix(@angular-devkit/build-angular): use translation file in bundle hash calculations - #16817

Merged
mgechev merged 1 commit into
angular:masterfrom
clydin:localize-hashing
Feb 4, 2020
Merged

mgechev merged 1 commit into
angular:masterfrom
clydin:localize-hashing

Conversation

@clydin

@clydin clydin commented Feb 2, 2020

Copy link
Copy Markdown
Member

This change ensures that any changes to translation files is represented in the output file names when output hashing is enabled. This prevents the situation where a translation file only change to an application would result in built files with no change in output name.

…sh calculations

This change ensures that any changes to translation files is represented in the output file names when output hashing is enabled.  This prevents the situation where a translation file only change to an application would result in built files with no change in output name.
@clydin clydin added the target: patch This PR is targeted for the next patch release label Feb 2, 2020
@clydin
clydin requested a review from alan-agius4 February 3, 2020 18:08
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Feb 4, 2020
@mgechev
mgechev merged commit 878e447 into angular:master Feb 4, 2020
@clydin
clydin deleted the localize-hashing branch February 4, 2020 16:04
@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 Mar 6, 2020
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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants