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.

Sign up for GitHub

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

Jump to bottom

streamline software episode #87

Merged
merged 1 commit into from
Dec 20, 2022
Merged

streamline software episode #87

merged 1 commit into from
Dec 20, 2022

Conversation

ewallace
Copy link
Collaborator

@ewallace ewallace commented Dec 6, 2022 •
edited
Loading

Edits to the software episode after #86.

  • edited questions/objectives/keypoints to reflect the episode content.
  • expand "what is research software" as that is a key point
  • rephrased the "Submit code to a reputable DOI-issuing repository" section as "Code is like data" because Data Management covers DOIs and this seemed a clearer way to put the argument. Addresses Addressing DOIs in software episode? #71

I'd appreciate another human giving this a look and checking?

@ewallace ewallace linked an issue Dec 6, 2022 that may be closed by this pull request
Addressing DOIs in software episode? #71
Closed
ameynert
ameynert approved these changes Dec 20, 2022
View reviewed changes
Copy link
Collaborator

@ameynert ameynert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good!

@ameynert ameynert merged commit 205adc4 into gh-pages Dec 20, 2022
@ewallace ewallace deleted the 2022-12_streamline branch December 20, 2022 14:00
zkamvar pushed a commit that referenced this pull request Aug 31, 2023
Merge pull request #87 from carpentries-incubator/2022-12_streamline
2576358 
streamline software episode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ameynert ameynert ameynert approved these changes

@MatthiasMimault MatthiasMimault Awaiting requested review from MatthiasMimault

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Addressing DOIs in software episode?
2 participants
@ewallace @ameynert