Skip to content

Conversation

@ecrupper
Copy link
Contributor

Closes go-vela/community#325

Closes go-vela/community#659

Building off the work done in #556, using repository events can help maintain parity between GitHub repos and their Vela equivalent.

@ecrupper ecrupper requested a review from a team as a code owner September 28, 2022 20:52
@ecrupper ecrupper self-assigned this Sep 28, 2022
@codecov
Copy link

codecov bot commented Sep 28, 2022

Codecov Report

Merging #706 (af8988b) into main (9aa91f7) will decrease coverage by 0.13%.
The diff coverage is 1.92%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #706      +/-   ##
==========================================
- Coverage   54.79%   54.66%   -0.14%     
==========================================
  Files         216      216              
  Lines       16046    16086      +40     
==========================================
+ Hits         8793     8794       +1     
- Misses       6869     6908      +39     
  Partials      384      384              
Impacted Files Coverage Δ
api/webhook.go 0.00% <0.00%> (ø)
scm/github/webhook.go 86.10% <100.00%> (+0.03%) ⬆️

cognifloyd
cognifloyd previously approved these changes Oct 11, 2022
Copy link
Member

@cognifloyd cognifloyd left a comment

Choose a reason for hiding this comment

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

LGTM

jbrockopp
jbrockopp previously approved these changes Nov 15, 2022
Copy link
Contributor

@jbrockopp jbrockopp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@wass3r wass3r left a comment

Choose a reason for hiding this comment

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

lgtm - one minor request

@ecrupper ecrupper dismissed stale reviews from jbrockopp and cognifloyd via 320e582 December 13, 2022 17:17
@ecrupper ecrupper merged commit 09e827e into main Dec 13, 2022
@ecrupper ecrupper deleted the fix/sync-default-branch branch December 13, 2022 20:44
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.

server: Automatically Disable Repos in Vela when a repo is archived. VELA_REPO_BRANCH does not follow current default repository branch

6 participants