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

Changes to demographics #21

Merged
merged 3 commits into from
Jun 13, 2018
Merged

Changes to demographics #21

merged 3 commits into from
Jun 13, 2018

Conversation

emmairwin
Copy link
Contributor

  • Removed transgender (this would be a separate question under Gender Identity)
  • Added Notes/Disclaimers about what the demographic list current is, and is not (out of scope)
  • Added Neurodiversity
  • Updated child care, to caregiver - to include eldercare.

* Removed transgender (this would be a separate question under Gender Identity)
* Added Notes/Disclaimers about what the demographic list current is, and is not (out of scope)
* Added Neurodiversity
* Updated child care, to caregiver - to include eldercare.
…t as part of this

* Updated leadership metrics based on June 6th conversation
* Added context for how we have described leadership
… that out as part of this"

This reverts commit 5211781.
@dicortazar
Copy link
Member

@emmairwin any comment on @aserebrenik potential additions?

@dicortazar
Copy link
Member

In addition to this, thanks a lot for the other PRs. However, I've seen that in some cases some of the PRs contain commits referenced in previous PRs.

A cleaner way to proceed is to create a fork, then a branch in your local repo. I've seen in your repo that you're working in this way for the rest of the PRs. This basically means that you're pulling the first PR change into all of the branches. Not a big deal in any case, but a bit of headache till I realized this was done in this way :).

Said this, I'll check how I can merge this in the most cleanest way, and then we can produce some guidelines to contribute, either through the GitHub interface or CLI. Hope this works for you!

Thanks!!!

@dicortazar
Copy link
Member

I've open a ticket with @aserebrenik comments and will merge this one.

@dicortazar dicortazar merged commit 4db888a into chaoss:master Jun 13, 2018
dicortazar added a commit that referenced this pull request Jun 13, 2018
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.

None yet

3 participants