- 
                Notifications
    
You must be signed in to change notification settings  - Fork 4
 
chapter 2, addition of Publishing section #7
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
base: main
Are you sure you want to change the base?
Conversation
| 
           Other than the two comments above, this looks great to me. I didn't spot any typos. @ryterdal if you want to make changes, any changes you make on your fork (on the same branch) should automatically show up here. I think @Cephalopd is the reviewer for this topic, so I've also added her as a reviewer on this PR.  | 
    
          
 @nick-ulle What do you mean they'll automatically show up? Meaning I don't have to do another pull request?  | 
    
          
 Right, after you make the commits, they should show up automatically in this pull request.  | 
    
| 
           If you want to start working on a different topic before this PR is merged, I recommend doing it on another branch (or another fork).  | 
    
Bolded "open data", fixed margin, note issue.
Fixing {note} again in Publish Open Data section
    | 
           @Cephalopd Just an FYI that I'm actually continuing to make edits to this section because I'm planning to change the structure a bit and put the secondary practices in Chapter 2, as well. So I'll ping you again when it's actually ready for review.  | 
    
Adding Publishing section to end of Chapter 2 with open access publishing for open data and open articles, plus a section on open licenses.
Can you please merge these changes in?