Lower the number of required approvers for CRs to 1

opensearch-project / logstash-output-opensearch

A Logstash plugin that sends event data to a OpenSearch clusters and stores as an index.

Apache License 2.0

104 stars 80 forks source link

Closed dblock closed 6 months ago

dblock commented 6 months ago

Is your feature request related to a problem? Please describe.

Currently PRs require 2 PR to be merged. This is slowing us down with only 1-2 active maintainers.

Describe the solution you'd like

Lower the number of approvers to 1.

Additional context

PRs like #237 sitting out here for weeks.

dblock commented 6 months ago

@asifsmohammed @dlvenable @oeyh?

dlvenable commented 6 months ago

I agree with this.

oeyh commented 6 months ago

I agree.

dblock commented 6 months ago

Looks like the majority of maintainers agree, I lowered the number of approvers required to 1.