Skip to content

[BREAKING] normalizer: Rename optional parameter "translator" #96

Merged
RandomByte merged 3 commits into
masterfrom
normalizer-t8r-params
Jan 9, 2019
Merged

[BREAKING] normalizer: Rename optional parameter "translator" #96
RandomByte merged 3 commits into
masterfrom
normalizer-t8r-params

Conversation

@RandomByte
Copy link
Copy Markdown
Member

@RandomByte RandomByte commented Jan 9, 2019

This should future-proof the normalizer API to allow for eventually passing
translator objects/instances instead of a simple string.

BREAKING CHANGE: Renamed parameter "translator" of functions generateDependencyTree and generateProjectTree to "translatorName"

Downstream adaptions: SAP/ui5-cli#112

This should future-proof the normalizer API to allow for eventually passing
translator objects/instances instead of a simple string.

BREAKING CHANGE: Renamed parameter "translator" of functions generateDependencyTree and generateProjectTree to "translatorName"
argument

Parameters now need to be passed in the options object.

No breaking change as the static translator is private API.
@RandomByte RandomByte requested a review from matz3 January 9, 2019 21:20
RandomByte added a commit to SAP/ui5-cli that referenced this pull request Jan 9, 2019
@RandomByte RandomByte changed the title [INTERNAL] README: Fix JSDoc links [BREAKING] normalizer: Rename optional parameter "translator" Jan 9, 2019
RandomByte added a commit to SAP/ui5-cli that referenced this pull request Jan 9, 2019
@RandomByte RandomByte merged commit d069cdd into master Jan 9, 2019
@RandomByte RandomByte deleted the normalizer-t8r-params branch January 9, 2019 23:44
d3xter666 pushed a commit to UI5/cli that referenced this pull request Sep 24, 2025
d3xter666 pushed a commit to UI5/cli that referenced this pull request Sep 25, 2025
d3xter666 pushed a commit to UI5/cli that referenced this pull request Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants