Skip to content
This repository has been archived by the owner on Oct 23, 2021. It is now read-only.

A single project root directory #561

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

A single project root directory #561

wants to merge 28 commits into from

Conversation

benkeser
Copy link
Collaborator

The purpose of this PR is to re-configure here-based file paths to only have a single project root. Heretofore, each subdirectory has had it's own separate root, which has caused some issues/confusion. In this PR:

  • the correlates_reporting/ folder is always considered the project root and all file paths are references w.r.t. this folder.
  • I removed all .here files and replaced with the recommended here::i_am root finding approach
  • I updated all file paths in all subdirectories and verification folders to point to the right locations
  • All .Rmd files also ensure that the root.dir of the R session is also correlates_reporting/. This will ensure consistent behavior whether .Rmd files are knitted in correlates_reporting/ or in subdirectories.

@benkeser benkeser marked this pull request as draft September 16, 2021 13:23
benkeser added a commit that referenced this pull request Sep 16, 2021
benkeser added a commit that referenced this pull request Sep 16, 2021
benkeser added a commit that referenced this pull request Sep 16, 2021
benkeser added a commit that referenced this pull request Sep 16, 2021
benkeser added a commit that referenced this pull request Sep 16, 2021
benkeser added a commit that referenced this pull request Sep 17, 2021
benkeser added a commit that referenced this pull request Sep 17, 2021
benkeser added a commit that referenced this pull request Sep 20, 2021
benkeser added a commit that referenced this pull request Sep 20, 2021
benkeser added a commit that referenced this pull request Sep 20, 2021
benkeser added a commit that referenced this pull request Sep 20, 2021
benkeser added a commit that referenced this pull request Sep 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant