Skip to content

Add GHA - #27

Open
hfrick wants to merge 1 commit into
mainfrom
add-gha
Open

Add GHA#27
hfrick wants to merge 1 commit into
mainfrom
add-gha

Conversation

@hfrick

Copy link
Copy Markdown
Collaborator

from https://github.com/cran-task-views/ctv/tree/main/validate-ctv

via https://github.com/cran-task-views/ctv/blob/main/Maintenance.md#task-view-updates

@zeileis I ran into this when trying to open the PR directly from my IDE (Positron with the GH extension). Before I dive into details, are you generally open to changing access permissions here?

Although you appear to have the correct authorization credentials, the cran-task-views organization has enabled OAuth App access restrictions, meaning that data access to third-parties is limited. For more information on these restrictions, including how to enable this app, visit https://docs.github.com/articles/restricting-access-to-your-organization-s-data/

@zeileis

Copy link
Copy Markdown
Contributor

Hannah @hfrick, I'm sorry I don't know what this means exactly. To me your PR looks reasonable. Maybe Nathalie @tuxette knows better what the problem might be here.

In any case, I don't recall that we discussed OAuth App access restrictions explicitly. I guess this never came up so far.

@tuxettetuxette self-assigned this Jun 12, 2025
@tuxette

Copy link
Copy Markdown

I'll check this this week-end at the latest.

@tuxette

Copy link
Copy Markdown

Hi @hfrick ! I checked it and indeed organization settings include OAuth App access restriction. This seems reasonable for us to keep it this way but we should be able to allow the access specifically for your IDE (we have Travis access enabled at the current time)... but I don't find a way to do it! Do you have an idea of how this can be done for positron or for a specific instance of positron?

@hfrick

Copy link
Copy Markdown
CollaboratorAuthor

Hi @tuxette , thanks for the feedback! I read up on the docs and you can't approve until I've sent a request. I've done that now and the GitHub docs suggest that you'll receive a notification of that request!

You can approve it as laid out here https://docs.github.com/en/organizations/managing-oauth-access-to-your-organizations-data/approving-oauth-apps-for-your-organization

Thank you!

@tuxette

Copy link
Copy Markdown

Thanks Hannah @hfrick ! It should be OK now.

Sign up for freeto 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

@hfrick@zeileis@tuxette