JBIDE-22446 use central.tpc.version = 4.60.0.Final-SNAPSHOT

[nickboldt]

JBIDE-22446 use central.tpc.version = 4.60.0.Final-SNAPSHOT -- should have made this change 4 sprints ago!

[alexeykazakov]

As you can see in the PR check above this PR doesn't solve the problem:
[ERROR] Found property angularjs.repo.url = http://download.jboss.org/jbosstools/targetplatforms/jbtearlyaccesstarget/4.60.0.Final-SNAPSHOT/REPO/

It's Final, not Alpha now but it's still a snapshot.

[nickboldt]

Re run the PR check using the latest parent pom and it should now work.

On Aug 3, 2016 8:18 PM, "Alexey Kazakov" notifications@github.com wrote:

As you can see in the PR check above this PR doesn't solve the problem:
[ERROR] Found property angularjs.repo.url =
http://download.jboss.org/jbosstools/targetplatforms/jbtearlyaccesstarget/4.60.0.Final-SNAPSHOT/REPO/

It's Final, not Alpha now but it's still a snapshot.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#567 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAN5DUPGBT-b5ZSf_H7_i_uqOnt48UPcks5qcS_ogaJpZM4JcASz
.

[alexeykazakov]

Still failing.

[psrna]

testPR

[nickboldt]

PR works when I run it locally. shrug

[psrna]

I've configured pr check job which runs JST Integration Tests.

[alexeykazakov]

What is CDI11-Integration-tests check about?

[nickboldt]

Here are the tests run for CDI11-integration:

https://dev-platform-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/jst.prcheck/jdk=jdk1.8,label=rhel7/lastSuccessfulBuild/artifact/jbosstools-integration-tests/tests/org.jboss.tools.jst.ui.bot.test/target/surefire-reports/org.jboss.tools.jst.ui.bot.test.AllJSTTests.txt

maybe the job needs to be tweaked to set a better context/label when commenting in github?

[alexeykazakov]

CDI11-Integration-tests is a weird name for JST tests ;)

[nickboldt]

Yes. Curiously, the link for Details on "JST-Integration-tests" goes to the same job.

[nickboldt]

Opened a new PR #568 but it still has the CDI11-Integration-tests notation there too. Not sure how to get rid of that for future PRs, since the jst.prcheck job now only has the "JST-Integration-tests" context.