[FEATURE] Configurable output folder
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 65/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- cli, performance, testing
Línea de trabajo
Comienza con packages/cli/src/constants.ts y packages/measure/src/config.ts; después, sigue cómo se consumen RESULTS_DIRECTORY, RESULTS_FILE, BASELINE_FILE y REASSURE_OUTPUT_FILE. Revisa las pruebas o los puntos de entrada existentes de CLI y measure para comprobar la configuración del entorno. Se considera terminado cuando se pueden configurar carpetas de salida separadas por paquete y ambos comandos conservan el comportamiento predeterminado de .reassure.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
We are configuring reassure in a monorepo containing many packages. We configured it in multiple packages. Each package has its own particularities and jest configuration, meaning they each have their respective reassure command.
We run these commands from the root of our monorepo. The problem is that since there's no way to configure an output folder, each package competes with the same .reassure/ folder at the root, making it difficult to run reassure across packages.
Let's say we change a shared util and want to measure its impact across the monorepo. We can't run the baseline, make the change, and then compare because the files in the .reassure/ folder get overridden after each reassure run in the different packages.
Describe the solution you'd like
Adding an environment variable called REASSURE_OUTPUT_FOLDER that works on the cli command and the measure packages.
The cli change might be as simple as changing a few lines here:
to something like
export const RESULTS_DIRECTORY = process.env.REASSURE_OUTPUT_FOLDER || '.reassure';
export const RESULTS_FILE = `${RESULTS_DIRECTORY}/current.perf`;
export const BASELINE_FILE = `${RESULTS_DIRECTORY}/baseline.perf`;
For the measure package, it might conflict with the REASSURE_OUTPUT_FILE environment variable, requiring more refactoring.
Describe alternatives you've considered
We could also double down on the REASSURE_OUTPUT_FILE environment variable and make it work with the cli package.
Additional context
Such a configuration would allow us to set up a different output folder per package in our monorepo.
There might already be a way of doing this, but I couldn't find it. Let me know what you think and how I can help. I can even make a pull request if needed.
- Lenguaje dominante
- TypeScript
- Estrellas
- 1.5k
- Forks
- 39
- Merge medio
- 3 h 48 min
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 callstack/reassure
-
unstable (bi-modal) render durationsQuizá libre de nuevo @mdjastrzebski la tomó hace 150 días y no hay ningún pull request abierto. Abiertobug
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
-
[BUG] 0.0 ms render durationAbiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 32/100
-
[BUG] Report results not stableAbiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Todos los issues de callstack/reassure
Issues similares
-
status: needs triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
mastra-ai/mastra#29706 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
lingdojo/kana-dojo#33802 · 1 comentario · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
cloudflare/workerd#8290 ·
Los mantenedores suelen responder en 1 día
-
area: build-and-ci possible-duplicate S3: minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
manaflow-ai/cmux#38889 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
ValentinH/react-easy-crop#670 ·