Skip to content

Wanted to add myself to contributors list #88

Closed
wants to merge 1 commit into from

3 participants

@inetbiz
inetbiz commented Nov 3, 2013

No description provided.

@stephenyeargin
Raven Tools member

This actually needs to be expanded to include other folks mentioned in the change log as well.

See https://github.com/inetbiz/schema-creator/blob/43b4f4284d4ce781509fbd67a9cbd360fce192d3/readme.md#changelog

@SleeplessByte

I reckong @norcross didn't consider people a contributor unless they did a substantial amount of work.

@stephenyeargin
Raven Tools member

You're probably right, but it makes sense to me to encourage contributions to the project by way of simple kudos. All of you guys are doing great work in your free time for this open source project, and it is the least we (raventools) can do to say "thanks".

I mean, this graph already tells the "true" story. We could just link to that in the README. Of course by that measure, raventools shouldn't even be on the list. ;-)

@SleeplessByte

@stephenyeargin Haha, yes. I guess that's correct. (Speaking of which @inetbiz , could you add a commit and change @Derkje-J to @SleeplessByte) ?

@stephenyeargin
Raven Tools member

I think I've spotted the crux of it. See: http://wordpress.org/plugins/about/readme.txt and http://codex.wordpress.org/Writing_a_Plugin . The .txt version of it has to be in a specific format for our hosting the source code on WordPress Plugin Directory (namely, it doesn't use the GitHub username, but instead your other account on that site).

I think we should treat these two files as separate concerns (and it's a shame WordPress didn't call it something else or offer an alternative, such as package.xml, etc.). Added pull request.

@inetbiz
inetbiz commented Nov 3, 2013

@stephenyeargin Ok I think my user on wordpress.org is inetbizo

@SleeplessByte
@inetbiz
inetbiz commented Nov 3, 2013

Who's the maintainer? Who's going to update? Should I change my commit?

@stephenyeargin
Raven Tools member

In this case, I think we can close this pull request (without merge) and take a look at #89. The readme.txt file over there can be updated with the new identifiers.

@inetbiz inetbiz deleted the unknown repository branch Nov 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.