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

How to contribute to wiki pages #1344

Closed
tjfulle opened this issue May 18, 2017 · 21 comments
Closed

How to contribute to wiki pages #1344

tjfulle opened this issue May 18, 2017 · 21 comments
Labels
Framework tasks Framework tasks (used internally by Framework team)

Comments

@tjfulle
Copy link
Contributor

tjfulle commented May 18, 2017

@bartlettroscoe, how does one contribute to the wiki pages? It don't see a way to fork the wiki so that I can submit a PR. Nor do I see any create new page/edit page buttons. Should I just clone and send a diff to someone?

It seems valuable for new developer, such as myself, to give input to the wiki (at least to the new developer/building Trilinos pages) while the pain of ramping up is fresh on their mind...

@bartlettroscoe
Copy link
Member

@tjfulle,

GitHub really does not directly support a PR model for updating the wiki. But I prototyped how to do it with:

My recommendation was to create:

which is a clone of:

and then you can do a PR against the 'master' branch of the https://github.com/trilinos/TrilinosWiki.

If you want, I can do a review for you using my https://github.com/bartlettroscoe/TrilinosWiki repo. I can provide the exact git commands you need to run in order to do this if you are interested. Let me know.

CC: @trilinos/framework

@bartlettroscoe bartlettroscoe added the Framework tasks Framework tasks (used internally by Framework team) label May 18, 2017
@tjfulle
Copy link
Contributor Author

tjfulle commented May 18, 2017

Thanks @bartlettroscoe! I'll cloned https://github.com/trilinos/TrilinosWiki and I'll go from there.

I was going to add some information about building Trilinos with Mac OS X, but the wiki doesn't have a building page/tab to link from. Do you want to add a place-holder page for general build instructions that I can link off of?

@tjfulle
Copy link
Contributor Author

tjfulle commented May 18, 2017

Looks like https://github.com/trilinos/TrilinosWiki has a 404 error.

@jhux2
Copy link
Member

jhux2 commented May 18, 2017

@tjfulle I suspect @bmpersc or @jwillenbring needs to add you to the right permissions group.

@tjfulle
Copy link
Contributor Author

tjfulle commented May 18, 2017

@jhux2, ya, probably. When the switch is made to a PR model in #1155, contributing to the wiki might be something to think about?

@bmpersc
Copy link
Contributor

bmpersc commented May 18, 2017

I don’t think permissions are the problem. I get a 404 at that link too and I should have all the permissions. Is https://github.com/trilinos/Trilinos/Wiki the right url instead(Note the “/“ between trilinos and wiki.)? That one works for me, but I'm not sure that is the wiki you are talking about.

@bartlettroscoe
Copy link
Member

Sorry, for the confusion. I had proposed some time ago to create the GitHub repo:

and I prototyped this with a repo in my account:

The repository https://github.com/trilinos/TrilinosWiki does NOT yet exit. But I will go ahead and create it since I think that is the best workflow for proposing changes to the wiki.

@jhux2
Copy link
Member

jhux2 commented May 18, 2017

I don’t think permissions are the problem. I get a 404 at that link too and I should have all the permissions. Is https://github.com/trilinos/Trilinos/Wiki the right url instead(Note the “/“ between trilinos and wiki.)? That one works for me, but I'm not sure that is the wiki you are talking about.

@bmpersc, Tim doesn't see the edit button on the wiki. I have some vague memory that there's contributor write permissions ... but maybe I'm misremembering.

@tjfulle
Copy link
Contributor Author

tjfulle commented May 19, 2017

@jhux2 , I think you're right. I thought there was a wiki setting to allow more permissive editing, but I haven't been able to figure it out on my own repos. It would be a real hassle if the only way to contribute to the wiki is by PR, particularly if Trilinos adopts a restrictive PR model where relatively few developers have write permissions.

@bartlettroscoe
Copy link
Member

@tjfulle, I set up the GitHub repo https://github.com/bartlettroscoe/TrilinosWiki. Look at the README.md file that I created to explain how to use this to suggest changes to the wiki using PRs:

As an example, see my PR:

If you have any questions about that process let me know. I will add a section on how a @trilinos/framework team member can actually merge this back to the target Trilinos wiki repo:

It is pretty simple if you are comfortable with branches and multiple git repos.

Let me know how it goes.

@bartlettroscoe
Copy link
Member

@trilinos/framework,

Any desire to comment on:

?

Otherwise, I will merge and add to the wiki.

@bartlettroscoe
Copy link
Member

I added the section:

to the instructions and followed it to merge the branch in trilinos/TrilinosWiki#2 and close the PR.

I am now moving this into to review.

@bartlettroscoe bartlettroscoe added the stage: in review Primary work is completed and now is just waiting for human review and/or test feedback label May 23, 2017
@jhux2
Copy link
Member

jhux2 commented May 24, 2017

I added the section:

https://github.com/trilinos/TrilinosWiki/blob/master/README.md#reviewing-and-merging-pr-branch-to-trilinos-wiki

to the instructions and followed it to merge the branch in trilinos/TrilinosWiki#2 and close the PR.

I am now moving this into to review.

Boy, this sure seems like a lot of work, when it seems like there's something wrong with permissions on @tjfulle's account. Here's what I see when I try to edit the wiki:

trilinos-wiki

@tjfulle
Copy link
Contributor Author

tjfulle commented May 24, 2017

@jhux2 , there is nothing technically wrong with permissions, I am listed as a member of trilinos/Trilinos, but not a "contributor". But, it sure is a pain to contribute to a wiki, if non-contributors have to clone a different repository, create the markdown page, open a pull request, then a Trilinos contributor has to edit the new page to link it to the existing wiki site, ... It seems to defeat the purpose of a wiki. Unless there is a real danger of people maliciously sabotaging the wiki by posting bad information, I believe the wiki should be opened to allow contributions from any GitHub member.

@tjfulle
Copy link
Contributor Author

tjfulle commented May 24, 2017

@tjfulle said:

I believe the wiki should be opened to allow contributions from any GitHub member.

Just uncheck trilinos/Trilinos Settings > Restrict editing to collaborators only

@bmpersc
Copy link
Contributor

bmpersc commented May 24, 2017

@jhux2 @tjfulle I don't believe not being a collaborator is the issue. We don't have any setup right now. I did go poking around the membership lists and the best guess on why you don't have the edit button is that you are not in the "developers" team which is required to have write access to the Trilinos repos and I'm guessing the wiki. Let me look into if it is appropriate to add you to that team.

@bartlettroscoe
Copy link
Member

Unless there is a real danger of people maliciously sabotaging the wiki by posting bad information, I believe the wiki should be opened to allow contributions from any GitHub member.

I am not sure that is the best idea unless we can find a way for GitHub to send out email notifications when a wiki page is edited. (That is how Wikipedia is kept correct even if everyone is allowed to edit it. Without email edit notifications, I suspect that Wikipedia would be a complete mess.) But GitHub does not seem to directly support for email notifications when the wiki gets updated, as mentioned at:

But any Trilinos developer should obviously be able to edit the wiki without having to go through a PR.

But for larger suggested changes or additions from outside contirbutors, I think going through a PR model seems like it would be a reasonable thing to do.

@jhux2
Copy link
Member

jhux2 commented May 24, 2017

GitHub does not seem to directly support for email notifications when the wiki gets updated, as mentioned at:
https://webapps.stackexchange.com/questions/78567/how-to-receive-a-notification-when-someone-edits-my-projects-github-wiki

@bartlettroscoe That's too bad. But in that same link, using an RSS feed is mentioned. To see changes to the Trilinos wiki, here's the feed to subscribe to: https://github.com/trilinos/Trilinos/wiki.atom.

@tjfulle
Copy link
Contributor Author

tjfulle commented May 25, 2017

Thanks everyone for the help. I now have permissions to write to the Trilinos repo and, thus, the wiki. I think @bartlettroscoe has sufficiently documented what users would need to do to contribute to the wiki, so this issue could probably be closed.

FWIW, my opinion is that the wiki should be more open. I understand the points brought up as to why it is not (more open), but I believe the benefits outweigh the risks :)

@tjfulle
Copy link
Contributor Author

tjfulle commented Sep 6, 2017

@bartlettroscoe, feel free to close this issue, unless you think it warrants more action.

@bartlettroscoe
Copy link
Member

feel free to close this issue, unless you think it warrants more action.

Thanks. Closing as complete.

@bartlettroscoe bartlettroscoe removed the stage: in review Primary work is completed and now is just waiting for human review and/or test feedback label Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework tasks Framework tasks (used internally by Framework team)
Projects
None yet
Development

No branches or pull requests

4 participants