Skip to content

feat: update with generated token#32

Merged
Troejelsgaard merged 8 commits intomainfrom
Update_to_use_pat
Sep 5, 2025
Merged

feat: update with generated token#32
Troejelsgaard merged 8 commits intomainfrom
Update_to_use_pat

Conversation

@Troejelsgaard
Copy link
Copy Markdown
Contributor

@Troejelsgaard Troejelsgaard commented Sep 4, 2025

Update: Added option to use a GitHub App to generate a new token that have access to private/internal repositories. This token is then used when installing the R dependencies.

@akselthomsen
Copy link
Copy Markdown
Member

you beat me to it!
I will just check if this also works with my solution and get back to reviewing it 😄

Signed-off-by: Aksel Thomsen <thomsenaksel@gmail.com>
Signed-off-by: Aksel Thomsen <thomsenaksel@gmail.com>
Signed-off-by: Aksel Thomsen <thomsenaksel@gmail.com>
Signed-off-by: Aksel Thomsen <thomsenaksel@gmail.com>
Signed-off-by: Aksel Thomsen <thomsenaksel@gmail.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 5, 2025

Code coverage

Name Coverage (%)
r.workflows 100
R/hello.R 100

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 5, 2025

Github pages

Review the pkgdown webpage for the PR here

@akselthomsen akselthomsen changed the title update with ACTION_PAT feat: update with generated token Sep 5, 2025
@akselthomsen akselthomsen self-requested a review September 5, 2025 12:54
Copy link
Copy Markdown
Member

@akselthomsen akselthomsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Troejelsgaard if it also works for you we should get it in main 😄

@Troejelsgaard Troejelsgaard merged commit e650da0 into main Sep 5, 2025
13 of 14 checks passed
@Troejelsgaard Troejelsgaard deleted the Update_to_use_pat branch September 5, 2025 13:42
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.

2 participants