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

Adopt redesigned version of the lesson #887

Draft
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

tobyhodges
Copy link
Member

As detailed in #886 and https://carpentries.org/blog/2024/03/dc-r-ecology-adoption/, this will replace the current version of the lesson with the redesigned version created by @MCMaurer and approved by @datacarpentry/curriculum-advisors-ecology.

To be merged 2024-07-08

@tobyhodges tobyhodges added the status:wait Progress dependent on another issue or conversation label Mar 26, 2024
@tobyhodges tobyhodges self-assigned this Mar 26, 2024
Copy link

⚠️ WARNING ⚠️

This pull request contains a mix of workflow files and regular files. This could be malicious. No preview will be created.

regular files:

  • .gitignore
  • README.md
  • config.yaml
  • episodes/00-before-we-start.Rmd
  • episodes/01-intro-to-r.Rmd
  • episodes/02-starting-with-data.Rmd
  • episodes/03-dplyr.Rmd
  • episodes/04-visualization-ggplot2.Rmd
  • episodes/05-r-and-databases.Rmd
  • episodes/data/cleaned/surveys_complete_77_89.csv
  • episodes/data/cleaned/surveys_meanweight_species_plot.csv
  • episodes/data/generating_CSVs.R
  • episodes/data/new_data.zip
  • episodes/data/raw/plots_new.csv
  • episodes/data/raw/species_new.txt
  • episodes/data/raw/surveys_new.csv
  • episodes/fig/DC-logo-vision.png
  • episodes/fig/DC1-logo-small.png
  • episodes/fig/R-ecology-add-two-plots-example-1.png

workflow files:

  • .github/ISSUE_TEMPLATE/pilot_workshop_feedback.yml
  • .github/workflows/README.md
  • .github/workflows/pr-close-signal.yaml
  • .github/workflows/pr-comment.yaml
  • .github/workflows/pr-post-remove-branch.yaml
  • .github/workflows/pr-preflight.yaml
  • .github/workflows/pr-receive.yaml
  • .github/workflows/sandpaper-main.yaml
  • .github/workflows/sandpaper-version.txt
  • .github/workflows/update-cache.yaml
  • .github/workflows/update-workflows.yaml

fkohrt and others added 8 commits April 8, 2024 16:33
Remove the curly braces from the fenced
code block (curly braces indicate executable
chunks in knitr and Quarto).
The section on required packages referenced
a heading on the homepage that is outdated
Forcats 1.0.0 depreciated `fct_explicit_na`
Equivalent way is now `fct_na_value_to_level(sex, "(Missing)")`

Fixes/addresses #11
Fix code block and reference
fct_explicit_na deprecated using fct_na_value_to_level instead
Update data link to refer to current repo
manually update lesson dependencies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:wait Progress dependent on another issue or conversation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants