Skip to content
This repository was archived by the owner on Jan 25, 2024. It is now read-only.

Switch to external PR workflow only #571

Open
trickeydan opened this issue May 29, 2020 · 3 comments · Fixed by #573
Open

Switch to external PR workflow only #571

trickeydan opened this issue May 29, 2020 · 3 comments · Fixed by #573

Comments

@trickeydan
Copy link
Contributor

Would it be better to have everyone follow the same (less privileged) workflow? This could help us surface issues in things like CI that may not appear until someone outside the organisation try to contribute which would impeded us merging in there changes without some potentially unnecessary back and forth.

Originally posted by @sedders123 in #539

We should test the external PR as part of this issue, and update the documenation accordingly.

@trickeydan
Copy link
Contributor Author

Reopening until #609 gets closed and the branch deleted. This is the last branch on the main repo.

After that is merged, we should email the mailing list and restrict writes.

@sedders123
Copy link
Member

#609 looks to be merged, is this unblocked now?

@trickeydan
Copy link
Contributor Author

To Do:

  • Check that the docs reflect the new workflow
  • Email the j5 mailing list to ensure that contributors know
  • Lock down pushing of branches if possible

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants