forcedotcom / forcedotcom/code-analyzer

[Feature Request] Add Input Variable To Control PR Review Behavior

Abierto
#1,965 3 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
TypeScript
Estrellas
240
Forks
52
Merge medio
1 d 23 h
PR fusionados (30 d)
5

Descripción

### Product Area

The “Run Salesforce Code Analyzer” GitHub Action

### Your Need or Problem

Currently, if I want to leverage the new `num-sev{X}-violations-in-changed-files` output variables I have to provide a github token with read access. Doing so leads to the action submitting a review for my PR.

I don't want the action to leave a review, I would simply like to know if there are violations in my changed files so that I can take actions against that in a following step. This is a pain point.

### Your Desired Solution

I would like some sort of input variable that lets me disable the code review. This should default to false to ensure that no existing workflows are broken. This would allow me to make use of the newly introduced `*in-chaged-files` output variables _without_ having a a review submitted via the action itself.

Something along the lines of:

```yaml
disableReview:
description: 'Prevents the action from submitting a review even if the github token is provided'
required: false
type: 'boolean'
default: 'false'
```

### Alternatives Considered

_No response_

### Additional Context (Screenshots, Files, etc)

_No response_

### Workaround

_No response_

### Urgency

Moderate

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Empieza localizando las definiciones de entrada de la GitHub Action y la ruta que envía una revisión de pull request. Añade cobertura para la nueva entrada opcional, su valor predeterminado false y el caso en que las salidas de infracciones sigan disponibles sin enviar una revisión; ejecuta las pruebas existentes de la Action o las comprobaciones del workflow.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
github-actions, typescript
Área
ci-cd
Tipo de issue
Nueva funcionalidad
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.