-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IDE-3359: Add Mojos for transpile and config #53
Conversation
Pull request test invoked at http://test-1-20.liferay.com/job/test-portal-acceptance-pullrequest(master). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, did you try this in the IDE (after building locally) to verify that it works and fixes the problem?
<executions> | ||
<execution> | ||
<id>transpile-js</id> | ||
<id>transpile-js</id> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whitespace
</arguments> | ||
</configuration> | ||
<configuration> | ||
<nodePath>${project.build.directory}/node/node</nodePath> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets add Defaults for the following variables and remove them from the pom
bundleFileName
format
globalName
moduleName
soySrcs
soySrc
srcs
src
</execution> | ||
<execution> | ||
<id>config-js-modules</id> | ||
<id>config-js-modules</id> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whitespace
<nodePath>${project.build.directory}/node/node</nodePath> | ||
<workingPath>${project.build.outputDirectory}/META-INF/resources</workingPath> | ||
<modulePath>${node.modules.dir}/liferay-module-config-generator/bin/index.js</modulePath> | ||
<config></config> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also set defaults for these and remove them from configuration
config
extension
format
ignorePath
namespace
<execution> | ||
<id>replace-translation</id> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whitespace
@@ -5,6 +5,8 @@ | |||
<pluginExecution> | |||
<pluginExecutionFilter> | |||
<goals> | |||
<goal>transpile-js</goal> | |||
<goal>config-js-modules</goal> | |||
<goal>build</goal> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does the build
goal do? do we need it? it seems out of place now
The pull request tester is still running.Please wait until you get the final report before running 'ci:retest'. See this link to check on the status of your test: However, the pull request was closed.The pull request was closed because the following critical batches had failed: For information as to why we automatically close out certain pull requests see this article. *This pull will no longer automatically close if this comment is available. If you believe this is a mistake please re-open this pull by entering the following command as a comment. ci:reopen Critical Failure Details:test-portal-acceptance-pullrequest-batch(master)/source-format-jdk8Job Results:0 Tests Passed.
|
ci:stop |
No description provided.