Skip to content

Update setup.md#89

Merged
dcmcand merged 1 commit intoLibraryCarpentry:gh-pagesfrom
skramer-Y2K:patch-1
Jul 30, 2020
Merged

Update setup.md#89
dcmcand merged 1 commit intoLibraryCarpentry:gh-pagesfrom
skramer-Y2K:patch-1

Conversation

@skramer-Y2K
Copy link
Copy Markdown
Contributor

Changing of the default editor will apparently accept ANY argument, even if that then won’t work – for instance, it happily accepts:

git config --global core.editor "foobar"

too, but there’s no step for testing that in the instructions. That step has been added here.

Please delete this line and the text below before submitting your contribution.


Thanks for contributing! If this contribution is for instructor training, please send an email to checkout@carpentries.org with a link to this contribution so we can record your progress. You’ve completed your contribution step for instructor checkout just by submitting this contribution.

Please keep in mind that lesson maintainers are volunteers and it may be some time before they can respond to your contribution. Although not all contributions can be incorporated into the lesson materials, we appreciate your time and effort to improve the curriculum. If you have any questions about the lesson maintenance process or would like to volunteer your time as a contribution reviewer, please contact The Carpentries Team at team@carpentries.org.


Changing of the default editor will apparently accept ANY argument, even if that then won’t work – for instance, it happily accepts:

git config --global core.editor "foobar"

too, but there’s no step for testing that in the instructions. That step has been added here.
@dcmcand dcmcand merged commit 7be56cb into LibraryCarpentry:gh-pages Jul 30, 2020
@dcmcand
Copy link
Copy Markdown
Contributor

dcmcand commented Jul 30, 2020

@skramer-Y2K looks good. Thanks!

zkamvar pushed a commit that referenced this pull request Apr 21, 2023
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