Skip to content

Add OMEVisual#55

Merged
ctrueden merged 1 commit into
scijava:masterfrom
hadim:omevisual
Jan 22, 2018
Merged

Add OMEVisual#55
ctrueden merged 1 commit into
scijava:masterfrom
hadim:omevisual

Conversation

@hadim

@hadim hadim commented Jan 22, 2018

Copy link
Copy Markdown
Contributor

No description provided.

@ctrueden ctrueden merged commit a99643d into scijava:master Jan 22, 2018
@ctrueden

Copy link
Copy Markdown
Member

Thanks!

@hadim hadim deleted the omevisual branch January 22, 2018 12:34
@hadim

hadim commented Jan 22, 2018

Copy link
Copy Markdown
Contributor Author

Only you can release a new Scijava version. It's the same to merge the others PRs:

So no rush. Do it when you have time.

@ctrueden

ctrueden commented Jan 22, 2018

Copy link
Copy Markdown
Member

I cut a new release 19.1.0 of pom-scijava. Travis should build and deploy it soon. Please update the above PRs to use it (amend the commit) when you have time, and then I'll merge them.

@hadim

hadim commented Jan 22, 2018

Copy link
Copy Markdown
Contributor Author

I am not sure but I think the build fails without Travis complaining about it and so the new version didn't get updated to Maven: https://travis-ci.org/scijava/pom-scijava/builds/331953772

Also for fiji-javadoc the Scijava version is set as RELEASE. I guess I should let like that right? https://github.com/fiji/fiji-javadoc/blob/master/pom.xml#L11

@hadim

hadim commented Jan 22, 2018

Copy link
Copy Markdown
Contributor Author

Here is the error:

[INFO] [ERROR] Failed to execute goal org.sonatype.plugins:nexus-staging-maven-plugin:1.6.7:deploy (injected-nexus-deploy) on project pom-scijava: Execution injected-nexus-deploy of goal org.sonatype.plugins:nexus-staging-maven-plugin:1.6.7:deploy failed: 504 - Gateway Timeout -> [Help 1]

Seems to be a server connectivity issue...

@ctrueden

Copy link
Copy Markdown
Member

the build fails without Travis complaining about it

FYI, I figured out why that was happening. Now fixed: scijava/scijava-scripts@1486380

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.

2 participants