Make e2e tests mandatory for PR merging
Los mantenedores suelen responder en 3 días
@clairep94 ya está trabajando en esto.
Desde el 1/9/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Increasing Access
Unsure
Feature enhancement details
Context:
This issue is a followup from GSOC:
- @Geethegreat has create an e2e test suite for his GSOC 2026 project
- We purposely made it so that e2e tests were not required for merging PR's so that the GSOC project did not unnecessarily block other work on the repo
- We also purposely made it so that e2e tests were gated by approval from @Geethegreat, @clairep94, @khanniie, or @raclim during the GSOC period, so that we did not accidentally overspend on GHA hours
- Now that GSOC is complete, we should update so that the e2e test suite is mandatory for merging into
develop - Ideally we should update to:
- Remove the approval gating
- Have e2e tests trigger automatically after successful unit tests
- Add some
concurrencyrules to ensure this doesn't result in a lot of GHA hours useage- Cancel any in-progress runs if a later run is triggered
- Maybe debounce triggers if this is possible
- Ideally update to minimize the required amount of time for e2e tests to run:
- Make app startup in CI faster if possible
- Caching any installations (some work previously done here already)
--> Will self-assign this issue
Requirements:
- Access to repo settings > environments & branch protection
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.7k
- Forks
- 1.7k
- Merge medio
- 5 d 15 h
- PR fusionados (30 d)
- 9
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de processing/p5.js-web-editor
-
Renaming a folder doubles the folder name in file paths (console errors show /lib/lib/helper.js)AbiertoAwaiting Maintainer Approval Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
processing/p5.js-web-editor#4365 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
p5 logo is broken on verification emailsPosiblemente ocupada @sourabh65mehta la tomó hace 2 días. AbiertoBug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
processing/p5.js-web-editor#4364 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 3 días
-
`PATCH /editor/project/visibility` returns 200 with `null` and doesn't update when `projectId` is a slugQuizá libre de nuevo @Prbhtsgh la tomó hace 9 días y no hay ningún pull request abierto. AbiertoAwaiting Maintainer Approval Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
processing/p5.js-web-editor#4348 · 3 comentarios ·
Los mantenedores suelen responder en 3 días
-
Fix: example.js stores defaultHTML function reference instead of calling it, causing examples to display raw JavaScript source in previewQuizá libre de nuevo @syedbarkath980 la tomó hace 13 días y no hay ningún pull request abierto. AbiertoAwaiting Maintainer Approval Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
processing/p5.js-web-editor#4344 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
signup form gets stuck when signup request fails due to network errorPosiblemente ocupada @PS01K la tomó hace 43 días. AbiertoAwaiting Maintainer Approval Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
processing/p5.js-web-editor#4285 · 3 comentarios ·
Los mantenedores suelen responder en 3 días
Todos los issues de processing/p5.js-web-editor
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
bigskysoftware/htmx#4147 ·
Los mantenedores suelen responder en 5 días
-
Please add PNG images as wellAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
vshymanskyy/StandWithUkraine#316 ·
-
README: the green and red points are swapped (green = .fset tracking, red = .fset3 detection)Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
-
bug/1-unconfirmed kind/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día