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

Create github.R #301

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Create github.R #301

wants to merge 3 commits into from

Conversation

Ssunoo2
Copy link
Collaborator

@Ssunoo2 Ssunoo2 commented Apr 28, 2024

Script to call github refresh from the command line

@RavenMarQ RavenMarQ mentioned this pull request Dec 3, 2024
12 tasks
Copy link
Member

@carlosparadis carlosparadis left a 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

Copy link
Member

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

Copy link
Member

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)
Copy link
Member

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.
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