-
Notifications
You must be signed in to change notification settings - Fork 13
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
Create github.R #301
base: master
Are you sure you want to change the base?
Create github.R #301
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pointing out here the get and other files ahead of time, same issue as @daomcgill
.github/workflows/R-CMD-check.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure you revert this or sync to master
.github/workflows/test-coverage.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same on revert to master
exec/github.R
Outdated
save_path_issue_or_pr_comments <- paste0(save_path,"/issue_or_pr_comment/") | ||
# Path you wish to save all raw data. A folder with the repo name and sub-folders will be created. | ||
owner <- conf[["issue_tracker"]][["github"]][["owner"]] # Has to match github organization (e.g. github.com/sailuh) | ||
repo <- conf[["issue_tracker"]][["github"]][["repo"]] # Has to match github repository (e.g. github.com/sailuh/perceive) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hardcoded get
To standardize github.R to match mailinglist.R executable. Also updated implementation to utilize the configuration get functions.
Script to call github refresh from the command line