expressjs / expressjs/discussions

EFI: Standardize checks for PR

Open
#186 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
No language data
Stars
73
Forks
25
PR merge metrics
No merged PRs in 30d

Description

### Motivation

Almost all repositories under the Extended Express Organization are packaging and publishing projects in the same way. Leveraging actions, we should be able to implement the same quality and the same "control" on all projects for each contribution, way of contributing and release. This need to include all the tests we want to have (quality test, performance test, security test...) and all processes we judge meaningful

### Expectation

Standardize tests we want to have on all steps and way to contribute

### Implementation

Standardize processes for opening a PR and for releases
Create actions for the new standard
Implement actions in all repositories

### Status

Part: Technical

### Draft

> Security + Benchmark + Tests
Most - if not all - the libraries handled by the Express organization could use the same process around PR management.
Automatic testing, security checks and performance benchmarks need to be automatic to detect in advance any drifts
Standardize PR process
Use reusable GitHub actions

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.