Skip to content
This repository has been archived by the owner on Feb 9, 2021. It is now read-only.

ENTESB-2722 Update GAV of integration project #3

Closed
wants to merge 1 commit into from

Conversation

cunningt
Copy link
Contributor

Update GAV of integration project

@ge0ffrey
Copy link

+1 to merge (Edson or Mario will merge I presume, they asked me to look over these).

@kevinearls
Copy link

retest this please

1 similar comment
@kevinearls
Copy link

retest this please

@dvirgiln
Copy link
Contributor

dvirgiln commented Mar 2, 2015

Can be merged this PR?? I am waiting until it is merged to create 3 different PR for:

ENTESB-2692 fuse-bxms-integ SY module error
ENTESB-2722 Update GAV of integration project
ENTESB-2696 Switchyard Different behaviours depending the maven version

I've not create the PRs for avoiding conflicts with this PR.

@cunningt cunningt mentioned this pull request Mar 2, 2015
@mariofusco
Copy link
Contributor

I cannot merge this PR because it breaks all the 3 tests in org.drools.karaf.itest.KieSpringOnKarafIntegrationTest

I suspect the problem is in the features.xml file, but I am not 100% sure. Please make those tests to pass and I'll merge the PR.

@chirino
Copy link
Contributor

chirino commented Mar 2, 2015

retest this please

@chirino
Copy link
Contributor

chirino commented Mar 2, 2015

So why are we switching to a non SNAPSHOT version?

@kevinearls
Copy link

retest this please

@cunningt
Copy link
Contributor Author

cunningt commented Mar 2, 2015

@chirino : Don't we need to provide a ER tag for productization? If I'm mistaken and productization can take a SNAPSHOT-versioned build, then let's close this pull request out and just fix the GAV issues.

@chirino
Copy link
Contributor

chirino commented Mar 2, 2015

@cunningt lets keep the master branch on SNAPSHOTS. Once it working nicely and we are ready to build we can create a new branch/tag and update to a non-SNAPSHOT GAV.

@cunningt
Copy link
Contributor Author

cunningt commented Mar 2, 2015

@chirino Sounds fine to me, closing this out.

@cunningt cunningt closed this Mar 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants