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

Document how to use GitHub to change YAML policy records #12

Open
jpolka opened this issue May 10, 2018 · 10 comments
Open

Document how to use GitHub to change YAML policy records #12

jpolka opened this issue May 10, 2018 · 10 comments

Comments

@jpolka
Copy link
Member

jpolka commented May 10, 2018

Updated CONTRIBUTING.md with instructions on how to modify policy records. Does it look like the simplest solution, @dhimmel ?

@dhimmel
Copy link
Member

dhimmel commented May 10, 2018

Yeah, but people will look to README first, so if you want to have the instructions in CONTRIBUTING, have the README obviously link there.

@jpolka
Copy link
Member Author

jpolka commented May 10, 2018

Thanks for adding the reference! This workflow is going to be brutal for people unfamiliar to github and I think will significantly limit participation. Is there some way we can give people permission to commit directly to a master branch (by making them members of an organization?) and then I can handle the pull request from that copy back into this one? Of course this would mean the person doing this work needs to resolve any validation issues themselves.

It would be extra great if search were functional on that working copy of the repo (ie, if it were not a fork...) as this would make it easier for people to find files.

The best alternative I can see is inviting people to download the yml files, edit them in a text editor, and email them to me, which would make tracking contributions and resolving conflicts from simultaneous edits much more complicated and cumbersome. Let me know what you think....

@dhimmel
Copy link
Member

dhimmel commented May 10, 2018

So if anyone is overwhelmed by the git intricacies, they can post there desired changes in an issue. We can set the commit authorship so they still get credit for the changes. For example,

git commit --author "Daniel Himmelstein <[email protected]>"

I think there is an easy-ish way for people to make pull requests through the GitHub interface in a way that will avoid conflicts. I think if we made a short video showing the process that would really help.

@jpolka
Copy link
Member Author

jpolka commented May 10, 2018

So if anyone is overwhelmed by the git intricacies, they can post there desired changes in an issue. We can set the commit authorship so they still get credit for the changes. For example,

This definitely seems better than an email! But could get hairy if people are trying to make multiple simultaneous changes, or want to change just one value on top of someone else's version. How would we sort this out, and would it not be easier to just make them members (with write access to only the policies YAML files) and let them commit themselves?

@dhimmel
Copy link
Member

dhimmel commented May 10, 2018

would it not be easier to just make them members (with write access to only the policies YAML files)

I don't really think giving write access to only the policy files is possible (maybe it would be with a submodule, but it would not be elegant).

Conflicts should only occur when two individuals edit the same lines in the same files. Since we don't expect to be making big schema updates, I don't think conflicts will occur that frequently. We can mention in the documention to check for an open pull request for a policy before starting a new one. In general we should encourage one pull request per publisher/policy

Let's have a chat if you're around so we can figure out how to best make this project accessible for contributors.

@jpolka
Copy link
Member Author

jpolka commented May 10, 2018

I don't really think giving write access to only the policy files is possible

We could make a repo with ONLY the policies files, and provide them access to that?

I'm imagining one user editing a policy and leaving their edits in a comment on an issue. A second user comes around and wants to change one value. Presumably the best way for them to do this would be to copy & paste from the most recent comment and make their desired changes. But how would we then attribute the changes appropriately?

@jpolka
Copy link
Member Author

jpolka commented May 10, 2018

I believe @SamanthaHindle will test the instructions at CONTRIBUTING.MD from an unaffiliated account; then we should be ready to spread the word! @garymcdowell it would be great if you could give this a shot as well!

@garymcdowell
Copy link

garymcdowell commented May 10, 2018 via email

@dhimmel
Copy link
Member

dhimmel commented May 10, 2018

Is there a way to just click on the journal in the priority list to get to the YAML file?

No but journal names are lexigraphically sorted in this file. Hence you can more easily find the relevant line:

Development	2082	2015-11-16

So you'll want romeo_2082.yml

@garymcdowell
Copy link

garymcdowell commented May 10, 2018 via email

@dhimmel dhimmel changed the title Documentation Document how to use GitHub to change YAML policy records May 10, 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

No branches or pull requests

3 participants