Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding "data" emoji for collected data #334

Closed
smoia opened this issue Nov 20, 2019 · 17 comments
Closed

Adding "data" emoji for collected data #334

smoia opened this issue Nov 20, 2019 · 17 comments
Assignees
Labels
discussion Discussion relevant to the repo enhancement New feature or request status: accepted

Comments

@smoia
Copy link
Contributor

smoia commented Nov 20, 2019

Is your feature request related to a problem? Please describe.
I discovered all-contributors thanks to tedana, a package for neuroimaging data analysis. I'm trying to implement it for phys2bids, another package for neuroimaging related data handling.
I hope this is not too similar to #63 , but the thing is that "content" as specified now doesn't seem to cover those people who collect data and contribute to a project in that way.
Also, it might be useful to distinguish between data for testing and data for "consumption".

Describe the solution you'd like
It would be nice to either update the definition of "content" contributors, or to add a new entry for "data" contributor (that doesn't seem too bounded to text production).

Describe alternatives you've considered

Additional context
It would even be better to have two entries, one for generic "data" and one for "data for tests".

@smoia smoia added the enhancement New feature or request label Nov 20, 2019
@Berkmann18
Copy link
Member

I see where you're coming from.
What description for content would you rather see? (PR welcome).

@Berkmann18 Berkmann18 added discussion Discussion relevant to the repo status: waiting for feedback labels Nov 22, 2019
@smoia
Copy link
Contributor Author

smoia commented Nov 22, 2019

It's not necessary to change the description of content that there is now, I think it's useful to have an entry like that for websites or app development and maintenance project. What I'm suggesting is a different entry for data handling projects (visualisation, analysis, preprocessing, ...) that need to have data for testing, or projects with data collection or classification (creation/update of databases, manual labellings etc.) to show a different kind of contribution.
If there's agreement about it, I can start a PR!

Suggested emoji: 🔣 to be fair to all the kind of data or 💾 for the good ol' days!

@Berkmann18
Copy link
Member

🔣 seems representative of that.
And I see value in this new type.

@all-contributors/core What do y'all think?

@all-contributors all-contributors deleted a comment from allcontributors bot Nov 23, 2019
@smoia
Copy link
Contributor Author

smoia commented Nov 24, 2019

@Berkmann18 I can work on a PR in the upcoming days!

@Berkmann18
Copy link
Member

@smoia That would be amazing (don't forget there are 3 repos to update for changes like this ;)).

@smoia
Copy link
Contributor Author

smoia commented Nov 25, 2019

@Berkmann18 Ok, I just noticed it. I'm going to use your project manager PRs (e.g. #141 ) as a model .

@smoia
Copy link
Contributor Author

smoia commented Dec 2, 2019

@Berkmann18 sorry to bother you, but I'm trying to add the emojis in the files. I'm typing it in, but that's not enough? How do you add the emoji inside the md file other than :code: ?

@Berkmann18
Copy link
Member

@smoia You may need to copy-paste it from GH or any place that have the emoji icon.

@smoia
Copy link
Contributor Author

smoia commented Dec 2, 2019

I did - still getting the expression itself rather than the emoji.
Might be editor related. Which one did you use?

@smoia
Copy link
Contributor Author

smoia commented Dec 2, 2019

@Berkmann18 managed! I had to manually enter the unicode equivalent.
First PR ready!

@smoia
Copy link
Contributor Author

smoia commented Dec 2, 2019

Second PR ready - should I wait for a new tag/release of all-contributors before modifying the all-contributors-bot?

@Berkmann18
Copy link
Member

I'm glad to see that you managed to do it.
Since the bot relies on the CLI, it would be better to make sure the bot uses the latest version of the CLI.
That being said, you could always apply the relevant changes, submit a draft PR and once the new CLI release is out you could add the version update in your bot PR.

That way, you already have the core of the bot PR ready to reviewed (even tho GH disables reviews for draft PRs) without having to wait.

@smoia
Copy link
Contributor Author

smoia commented Dec 3, 2019

Oh ok, I see.
Should I re-open the PR as draft?
Also for the bot, don't I have to wait for a new release of all-contributors (I can open a draft in the meantime).

@Berkmann18
Copy link
Member

@smoia For the CLI/spec repo, no need.

Also for the bot, don't I have to wait for a new release of all-contributors (I can open a draft in the meantime).

You could wait, however, I think it would be better to already have a (draft) PR submitted so you just need to commit the version change. Alternatively, you can set it up locally and push the PR once the release comes in.

@smoia
Copy link
Contributor Author

smoia commented Jan 3, 2020

@Berkmann18 last PR ready, tested and corrected. After that, I'd say we close this issue!

I have a question though: I already started to use the "data" emoji adding it manually in my contributors table. I guess the bot will pick it up without any problem, right?

@Berkmann18
Copy link
Member

Berkmann18 commented Jan 4, 2020

@smoia It won't pick it up when you tell it to add a data contribution (unless you mean the test bot you set up with your changes) until the code is pushed and deployed.
However, it should be fine for the table generation bit as the bot uses the CLI for this.

@smoia
Copy link
Contributor Author

smoia commented Jan 4, 2020

I see - yes, obviously the code has to be merged first. I'll wait and see!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Discussion relevant to the repo enhancement New feature or request status: accepted
Projects
None yet
Development

No branches or pull requests

2 participants