Skip to content

Review developers section of the parent POM#15

Merged
sbesson merged 2 commits intoome:masterfrom
sbesson:developers
Dec 6, 2016
Merged

Review developers section of the parent POM#15
sbesson merged 2 commits intoome:masterfrom
sbesson:developers

Conversation

@sbesson
Copy link
Copy Markdown
Member

@sbesson sbesson commented Dec 5, 2016

Post components decoupling, this PR reviews the developers to use the following conventions discussed with @jburel and @joshmoore:

  • lists the team as the main developers (matching the behavior of webapps) and uses the ome-devel email
  • lists all contributors of the repository as contributors

Comment thread pom.xml Outdated
<contributor><name>Helen Flynn</name></contributor>
<contributor><name>David Gault</name></contributor>
<contributor><name>Alexander Görtz</name></contributor>
<contributor><name>Mark Hinerm</name></contributor>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should be Mark Hiner

Comment thread pom.xml
<picUrl>http://loci.wisc.edu/files/loci/images/people/curtis-2010.jpg</picUrl>
</properties>
<name>The OME Team</name>
<email>ome-devel@lists.openmicroscopy.org.uk</email>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe we want <url>, <organization>, <organizationUrl> too ?
(I assume the <id>'s optional.)

Copy link
Copy Markdown
Member Author

@sbesson sbesson Dec 6, 2016

Choose a reason for hiding this comment

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

Some of this is already defined in the organization section above in the POM but I can add minimal information.

Copy link
Copy Markdown
Member

@mtbc mtbc Dec 6, 2016

Choose a reason for hiding this comment

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

Aha, true! I wonder if we can simply put an <organization> up there to at least name the Open Microscopy Environment but either way this PR is fine to merge.

@mtbc
Copy link
Copy Markdown
Member

mtbc commented Dec 6, 2016

Aha, now found <organization> tag way below. 👍

@sbesson sbesson merged commit 2a4b38b into ome:master Dec 6, 2016
@sbesson sbesson deleted the developers branch December 6, 2016 10:33
@sbesson sbesson added this to the 5.4.0 milestone Dec 8, 2016
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.

3 participants