Skip to content

Supports <pathToBundleMapper> and <sourcePathToTargetMapper>#119

Merged
yumaoka merged 3 commits intoIBM-Cloud:masterfrom
yumaoka:rfe-118-regexpath
May 7, 2018
Merged

Supports <pathToBundleMapper> and <sourcePathToTargetMapper>#119
yumaoka merged 3 commits intoIBM-Cloud:masterfrom
yumaoka:rfe-118-regexpath

Conversation

@yumaoka
Copy link
Contributor

@yumaoka yumaoka commented May 3, 2018

Supports new GP maven configuration parameter and
for more flexible resource file path -
bundle ID mappings.

Supports new GP maven configuration parameter <pathToBundleMapper> and
<sourcePathToTargetMapper> for more flexible resource file path -
bundle ID mappings.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 50.557% when pulling be0444d on yumaoka:rfe-118-regexpath into cc5026a on IBM-Cloud:master.

@coveralls
Copy link

coveralls commented May 3, 2018

Coverage Status

Coverage decreased (-0.5%) to 50.557% when pulling 80ade08 on yumaoka:rfe-118-regexpath into cc5026a on IBM-Cloud:master.

@yumaoka yumaoka requested review from jugu, pgaglani1 and srl295 May 3, 2018 23:41
@yumaoka
Copy link
Contributor Author

yumaoka commented May 3, 2018

The coverage report does not include maven plugin test cases under it folder invoked through maven invoker plugin. I added test cases that covers the new feature in the maven plugin test cases.

For each `<regexMapper>` element, substring matching a Java regular expression pattern specified by
`<pattern>` value will be replaced with the value specified by `<replacement>` value.

The example below will removes `.properties` from a give path, then replace `/` with `.`.
Copy link
Contributor

Choose a reason for hiding this comment

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

typo replace removes with remove

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Also from a give path with from a given path

<sourcePathToTargetMapper>
<regexMapper>
<pattern>(.+)/en/([^/]+).json</pattern>
<replacement>$1/%LANG/$2.json</replacement>
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be %LANG%?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct - I fixed above. Thanks.

Copy link
Contributor

@jugu jugu left a comment

Choose a reason for hiding this comment

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

lgtm

@yumaoka yumaoka merged commit f1861fb into IBM-Cloud:master May 7, 2018
@yumaoka yumaoka deleted the rfe-118-regexpath branch May 7, 2018 18:41
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.

3 participants