Skip to content
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

Remove Kogito profile to productize required components #3541

Merged
merged 1 commit into from Jan 16, 2021

Conversation

almope
Copy link
Contributor

@almope almope commented Jan 14, 2021

Signed-off-by: Alberto Morales Pérez almorale@redhat.com

Thank you for submitting this pull request

JIRA: (https://issues.redhat.com/browse/RHPAM-3393)

Referenced Pull Requests:

Business Central: n/a

VS Code: n/a

How to retest this PR or trigger a specific build:
  • Retest PR: jenkins retest this
  • A full downstream build: jenkins do fdb
  • A compile downstream build: jenkins do cdb
  • A full production downstream build: jenkins do product fdb
  • An upstream build: jenkins do upstream

Signed-off-by: Alberto Morales Pérez <almorale@redhat.com>
@sonarcloud
Copy link

sonarcloud bot commented Jan 14, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@Ginxo Ginxo left a comment

Choose a reason for hiding this comment

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

Check my proposal, but in general terms LGTM

<id>kogito</id>
<activation>
<property>
<name>!productized</name>
Copy link
Contributor

Choose a reason for hiding this comment

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

@almope
Why don't we use kogitoProductized instead?

Copy link
Member

Choose a reason for hiding this comment

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

we need to build it in community default workflow too @Ginxo ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Ginxo adding a new property implies that for both community and product build we have to add that as a parameter. Previously the productized property avoids the plugin executions on productization builds while now we need in all kind of builds.

Copy link

@AntStephenson AntStephenson left a comment

Choose a reason for hiding this comment

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

LGTM, removing the profile entirely is a better solution since we no longer need to exclude this from product builds.

@almope almope changed the title build GWT code and package WAR for productization build Remove Kogito profile to productize required components Jan 15, 2021
@almope almope marked this pull request as ready for review January 15, 2021 13:13
@mareknovotny mareknovotny merged commit e7770ed into kiegroup:master Jan 16, 2021
vpellegrino pushed a commit to vpellegrino/kie-wb-common that referenced this pull request Jan 18, 2021
Signed-off-by: Alberto Morales Pérez <almorale@redhat.com>
vpellegrino pushed a commit to vpellegrino/kie-wb-common that referenced this pull request Jan 18, 2021
Signed-off-by: Alberto Morales Pérez <almorale@redhat.com>
bxf12315 pushed a commit to Cory-jbpm/kie-wb-common that referenced this pull request Feb 2, 2021
Signed-off-by: Alberto Morales Pérez <almorale@redhat.com>
almope pushed a commit to almope/kie-wb-common that referenced this pull request Feb 4, 2021
Signed-off-by: Alberto Morales Pérez <almorale@redhat.com>
mareknovotny pushed a commit that referenced this pull request Feb 5, 2021
Signed-off-by: Alberto Morales Pérez <almorale@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants