Skip to content

Conversation

johnsonlab
Copy link
Contributor

Edited FAQ to include workflowr/Beaker Browser method for direct sharing of entire site(s) using dat:// links.

Edited FAQ to include workflowr/Beaker Browser method for direct sharing of entire site(s) using dat:// links.
@jdblischak jdblischak mentioned this pull request Sep 1, 2017
@jdblischak
Copy link
Member

@johnsonlab Thank you very much for the PR!

My only two concerns are

  1. It is a bit verbose for an FAQ entry
  2. I feel too self-conscious to have phrases like "truly revelatory" in the documentation (it's flattering coming from a user, but it's to self-congratulating to be in the software's own documentation)

Here's my proposal:

  1. I'll merge your PR
  2. I'll edit your text and post my changes here so that you can see them

Since this is your first PR on GitHub, I figured this would be easier than having you update your PR by adding additional commits to the branch of your fork. Does this work for you?

@johnsonlab
Copy link
Contributor Author

I accept your proposal and appreciate your understanding given this is my first pull request. Please edit accordingly and I will match the tone and limit verbosity next time. Thanks!

@jdblischak jdblischak merged commit 564c134 into workflowr:dev Sep 7, 2017
@jdblischak
Copy link
Member

Great. Congrats on getting your first GitHub PR merged! I'm honored it was for one of my projects!

@johnsonlab
Copy link
Contributor Author

johnsonlab commented Sep 8, 2017 via email

@jdblischak
Copy link
Member

I restructured the FAQ in commit 386a0e2. It'll be live in the online docs after the next release.

Cross-referencing Issue #59 here for future reference.

I look forward to your next PR!

@jdblischak
Copy link
Member

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.

2 participants