Skip to content
This repository has been archived by the owner on Apr 19, 2023. It is now read-only.

feat: reload config on push to .github repository, push new config all org repositories #26

Merged
merged 8 commits into from
May 2, 2022

Conversation

mrparkers
Copy link
Contributor

@mrparkers mrparkers commented Apr 27, 2022

here is a summary of the intended changes:

  • whenever the config file located within the .github folder changes, a new webhook event is sent to the app to signal that the config needs to be refreshed. previously, we'd refresh the config every 15 minutes, so there's a short window where the config would be updated but it wouldn't be reflected on the app
  • whenever the branch protection config changes, the app should push these changes to all non-whitelisted repositories in the github org. previously, these kinds of updates after a config change had to be manual

@liatrio-bot
Copy link

Terraform Plan: 0 to add, 1 to change, 0 to destroy.

Resources to update:

! azurerm_app_service.branch_protection_service

@liatrio-bot
Copy link

Terraform Plan: 0 to add, 1 to change, 0 to destroy.

Resources to update:

! azurerm_app_service.branch_protection_service

@liatrio-bot
Copy link

Terraform Plan: 0 to add, 1 to change, 0 to destroy.

Resources to update:

! azurerm_app_service.branch_protection_service

@liatrio-bot
Copy link

Terraform Plan: 0 to add, 1 to change, 0 to destroy.

Resources to update:

! azurerm_app_service.branch_protection_service

@liatrio-bot
Copy link

Terraform Plan: 0 to add, 1 to change, 0 to destroy.

Resources to update:

! azurerm_app_service.branch_protection_service

@mrparkers mrparkers marked this pull request as ready for review April 29, 2022 14:40
src/util/batch.js Outdated Show resolved Hide resolved
steve-liatrio
steve-liatrio previously approved these changes Apr 29, 2022
@liatrio-bot
Copy link

tf/tfplan.json

Terraform Plan: 0 to be created, 0 to be deleted, 1 to be updated, 0 to be replaced, 3 unchanged.

Resources to update

! azurerm_app_service.branch_protection_service

Copy link

@RobertKelly RobertKelly left a comment

Choose a reason for hiding this comment

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

Like the all repos feature as well.

@mrparkers mrparkers merged commit 9422055 into main May 2, 2022
@mrparkers mrparkers deleted the config-reload branch May 2, 2022 14:37
liatrio-bot pushed a commit that referenced this pull request May 2, 2022
## [1.2.0](v1.1.0...v1.2.0) (2022-05-02)

### Features

* reload config on push to .github repository, push new config all org repositories ([#26](#26)) ([9422055](9422055))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants