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

Permit additional users to commit to master #17

Closed
Lnaden opened this issue Aug 11, 2017 · 7 comments
Closed

Permit additional users to commit to master #17

Lnaden opened this issue Aug 11, 2017 · 7 comments

Comments

@Lnaden
Copy link

Lnaden commented Aug 11, 2017

Reminder issue per call today to allow at least @jchodera write access to this repo, and possibly me (@Lnaden) to make managing the site less burdensome on a few people. Also allows easier example additions.

@jchodera
Copy link
Member

I'd also like some credentials to access the openmm.org S3 bucket in case I have to do some manual cleanup.

I want to get this up before my talk on developing with OpenMM at the Tue 15 Aug at the E-CAM meeting. Can someone give me access to the repo and S3 ASAP?

@peastman
Copy link
Member

Done. We should still make changes through pull requests, though, so people can review them first.

I'd also like some credentials to access the openmm.org S3 bucket

I don't know who manages that. Maybe ask Theresa?

@jchodera
Copy link
Member

I think it was originally @rmcgibbo

@jchodera
Copy link
Member

jchodera commented Aug 14, 2017

Done.

Thanks!

We should still make changes through pull requests, though, so people can review them first.

To clarify, I think we should still make changes to master through pull requests from preview, but that it will be useful to make changes to preview that would allow us to review a live version of the page and make changes.

After merging #19, I think we should

  • push a copy of master to preview
  • protect the master branch from direct commits

@Lnaden
Copy link
Author

Lnaden commented Aug 14, 2017

I'm +1 on continuing changes by PR into preview and locking master from direct commits.

@jchodera
Copy link
Member

jchodera commented Aug 15, 2017

I've been given write access to the repo (and I think @Lnaden too).

@Lnaden
Copy link
Author

Lnaden commented Aug 15, 2017

Im a collaborator at the very least, so I should be good

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