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

Support repo creation as part of rill deploy by requesting Github write access #4488

Merged
merged 14 commits into from
Apr 9, 2024

Conversation

k-anshul
Copy link
Member

@k-anshul k-anshul commented Apr 2, 2024

Changes in Github App Permissions for Repository :

  1. New
image
  1. Updated from Read only to Read-Write
image

@k-anshul k-anshul self-assigned this Apr 2, 2024
@k-anshul k-anshul requested a review from begelundmuller April 3, 2024 06:27
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/pkg/cmdutil/prompt.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
cli/cmd/deploy/deploy.go Outdated Show resolved Hide resolved
@begelundmuller begelundmuller merged commit 7c789f8 into main Apr 9, 2024
7 checks passed
@begelundmuller begelundmuller deleted the github_auto_repo branch April 9, 2024 12:07
AdityaHegde pushed a commit that referenced this pull request Apr 9, 2024
…te access (#4488)

* interim commit

* github flow

* lint and fmt errors

* lint and fmt errors

* small fix

* grammar fix

* login and connect to github in one flow

* small error change

* review comments

* some more review comments

* comment change

* small msg change
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