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.

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

add link to git-rstudio-basics tutorial #164

Merged
merged 1 commit into from
Jan 30, 2024
Merged

Conversation

avallecam
Copy link
Member

  • The post specifies a license if you don't want to use the default CC BY
  • All authors have an ORCID iD
  • Relevant keywords / tags has been added. In particular, if you want your post to be shared on R-bloggers, you must tag it with R
  • Images or other external resources have been committed and pushed
  • The post uses pure quarto syntax, rather than HTML or R code, unless necessary

Right before merging:

  • The date field has been updated
  • All reviewers have been acknowledged in a short paragraph
  • A PR has been opened in the blueprints to link to this post
  • The post has been re-rendered and content of the _freeze/ folder is up-to-date

Copy link

netlify bot commented Jan 16, 2024

Deploy Preview for tourmaline-marshmallow-241b40 ready!

Name Link
🔨 Latest commit 0a553da
🔍 Latest deploy log https://app.netlify.com/sites/tourmaline-marshmallow-241b40/deploys/65b8b71c33997000084704dc
😎 Deploy Preview https://deploy-preview-164--tourmaline-marshmallow-241b40.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@avallecam avallecam requested a review from Bisaloo January 16, 2024 11:47
@avallecam
Copy link
Member Author

Hi @Bisaloo. This is an update to add the link to the git-tutorial. I have some questions given the template bullets as checkbox in the first comment:

  • do the date needs to be updated?
  • do I need to rerender the content?

I'll be happy to proceed with any additional required action for this update.

@Bisaloo
Copy link
Member

Bisaloo commented Jan 16, 2024

Hi Andree, thanks for this. No, this is ready to be merged as is. Flagging that this change will likely be made obsolete by #157 though.

I expect this content to be part of the intro to the lesson materials instead. What do you think?

@chartgerink chartgerink merged commit 74ccd19 into main Jan 30, 2024
6 checks passed
@chartgerink chartgerink deleted the git-training-patch-1 branch January 30, 2024 08:48
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.

3 participants