Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@AshFungor
Copy link
Collaborator

In progress :)

@AshFungor AshFungor marked this pull request as draft May 28, 2025 14:12
@AshFungor
Copy link
Collaborator Author

Hi @riebl, can you approve me as contributor for allowing workflow runs? :)
Thanks!

@AshFungor
Copy link
Collaborator Author

Looks like build & push needs additional permissions to update the runner image. I think it might be a bad practice to update this image globally (i.e. effectively every new PR will push & pull image from the same place), but, unfortunately, I see no other way we may implement this conveniently.

@riebl, may you grant package update permission?

@riebl
Copy link
Owner

riebl commented Jul 16, 2025

I have already checked that the package is linked to this repository. Not sure yet how to solve this issue properly.

@AshFungor
Copy link
Collaborator Author

image

Not sure if this helps, but this worked for me:

  • go to packages page, then open package settings
  • set permissions to write

(on picture is my fork)

@riebl
Copy link
Owner

riebl commented Jul 16, 2025

Yes, I have the same settings here. Maybe it is a limitation when a pull request triggers an action, but I don't know for sure.

@AshFungor
Copy link
Collaborator Author

Yeah, I think my branch is foreign to your repository and does not have write permissions :)

@riebl
Copy link
Owner

riebl commented Jul 16, 2025

I will separate the container creation from the CI: Build container only on push to master branch, run CI on push and pull requests. Then no write access is required when running CI for pull requests.

@riebl
Copy link
Owner

riebl commented Jul 16, 2025

Okay, the described change is live :-) Please rebase your PR and let's see what will happen!

@AshFungor AshFungor force-pushed the python-bindings branch 3 times, most recently from fd16fd9 to 3d74d5f Compare July 16, 2025 23:54
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