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

[FIX] update DECISION-MAKING.md document with new governance #441

Merged
merged 1 commit into from
May 6, 2020

Conversation

sappelhoff
Copy link
Member

This PR is an update to our DECISION-MAKING document, recognizing the current governance we are following.

I left the general GitHub workflow in the document ... but changed its name from "decision making" to "workflow". This PR is meant as a first step to improve the current state.

@effigies @franklin-feingold I added you under the Maintainer Group, I don't know your time commitments, so I left it the same as mine (~5h/week), which is a very rough estimate anyway (some weeks more, some weeks less).

|--------------------------------------------------------------------------------|-----------------|
| Stefan Appelhoff ([@sappelhoff](https://github.com/sappelhoff)) | 5h/week |
| Chris Markiewicz ([@effigies](https://github.com/effigies)) | 5h/week |
| Franklin Feingold ([@franklin-feingold](https://github.com/franklin-feingold)) | 5h/week |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
| Franklin Feingold ([@franklin-feingold](https://github.com/franklin-feingold)) | 5h/week |

I am working on clarifying this - will submit a follow up PR when I do

@franklin-feingold
Copy link
Collaborator

thanks for putting together @sappelhoff ! LGTM other than my small comment

@yarikoptic
Copy link
Collaborator

FWIW, linkchecked failed to connect to nitrc, restarted (I think) that workflow

@sappelhoff sappelhoff added this to the 1.3.1 milestone Apr 21, 2020
@sappelhoff
Copy link
Member Author

@franklin-feingold can this be merged and you follow up on it with a separate PR?

@franklin-feingold
Copy link
Collaborator

@sappelhoff yep! sounds good

@sappelhoff sappelhoff changed the title ENH update DECISION-MAKING.md document with new governance [FIX] update DECISION-MAKING.md document with new governance May 6, 2020
@sappelhoff sappelhoff merged commit 1f28fe2 into bids-standard:master May 6, 2020
@sappelhoff sappelhoff deleted the up branch May 6, 2020 17:23
yarikoptic added a commit to yarikoptic/bids-specification that referenced this pull request May 19, 2020
* origin/master: (404 commits)
  [DOC] Auto-generate changelog entry for PR bids-standard#460
  Apply suggestions from code review
  label -> index
  drop _part-, introduce _split-
  [DOC] Auto-generate changelog entry for PR bids-standard#459
  [DOC] Auto-generate changelog entry for PR bids-standard#465
  fix table
  Update src/99-appendices/06-meg-file-formats.md
  [DOC] Auto-generate changelog entry for PR bids-standard#441
  inject _part into MEG spec
  update entity table
  FIX: clarify _part
  Apply suggestions from code review
  FIX: clarify participants tsv
  [DOC] Auto-generate changelog entry for PR bids-standard#457
  Update Release_Protocol.md
  add pdf steps for release protocol
  FIX: remove BESA from list of restricted keywords
  Remove trailing space
  Add reference to PDF on front page of specification
  ...

Conflicts:
	src/02-common-principles.md - had to meld with my previous wording etc.
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