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

Update get_involved.md to add BEP036: Phenotypic Data Guidelines #231

Merged
merged 3 commits into from
Jun 8, 2022
Merged

Conversation

ericearl
Copy link
Contributor

@ericearl ericearl commented Jun 7, 2022

Add BEP036: Phenotypic Data Guidelines. I also accidentally did this commit on the wrong repo. I'm so sorry... Though I'm glad a new file of that nature shouldn't be a big deal.

Add BEP036: Phenotypic Data Guidelines.
@ericearl ericearl self-assigned this Jun 7, 2022
Copy link
Member

@sappelhoff sappelhoff left a comment

Choose a reason for hiding this comment

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

that's a lot of "leads", even more than for "common ephys derivatives" -- and there experience shows (IMHO) that it's too many and practically, the group of "leads" is smaller.

if you insist though, we can merge this as is. I think I would consider making the team of leads smaller though. Just let me know.

Correction from Stefan.  Only needed the leads, not everyone.
Two names don't need just one comma, they need an "and".
@ericearl
Copy link
Contributor Author

ericearl commented Jun 7, 2022

that's a lot of "leads", even more than for "common ephys derivatives" -- and there experience shows (IMHO) that it's too many and practically, the group of "leads" is smaller.

if you insist though, we can merge this as is. I think I would consider making the team of leads smaller though. Just let me know.

Thank you @sappelhoff ! Great catch. I totally misread the header as expecting everyone and not just the leads.

@sappelhoff sappelhoff merged commit 570b44f into bids-standard:gh-pages Jun 8, 2022
@sappelhoff
Copy link
Member

Okay, thanks!

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