Skip to content

Comments

fix: Add E2E test artifacts to .gitignore#399

Merged
nikblanchet merged 3 commits intonikblanchet:mainfrom
pvpeuve:fix/gitignore-e2e-artifacts
Nov 27, 2025
Merged

fix: Add E2E test artifacts to .gitignore#399
nikblanchet merged 3 commits intonikblanchet:mainfrom
pvpeuve:fix/gitignore-e2e-artifacts

Conversation

@pvpeuve
Copy link
Contributor

@pvpeuve pvpeuve commented Nov 25, 2025

This PR adds entries to .gitignore to ignore E2E test artifacts left by bash scripts (test-incremental-analysis.sh and test-status-command.sh), preventing untracked files from cluttering local worktrees.

Closes #398

@nikblanchet nikblanchet self-requested a review November 26, 2025 10:44
@nikblanchet
Copy link
Owner

nikblanchet commented Nov 26, 2025

¡Gracias por la contribución! Los patrones de .gitignore están muy bien.

Bien visto lo del desfase de versión en package-lock.json - me recordaste que había olvidado sincronizar la versión en todos lados cuando lancé 1.1.0-beta.1. Ya he mergeado ese fix en #400, así que cuando hagas rebase de este PR, ese diff va a desaparecer. No tienes que hacer rebase - es solo una opción. Abajo hay instrucciones por si no sabes cómo hacerlo.

También dejé una sugerencia para un comentario aclaratorio en .gitignore. No sabía si editar un branch de tu fork directamente sería de buen gusto. 😅 ¡Avísame si prefieres que haga el cambio yo!


Cómo hacer rebase de tu branch

Tu branch es parte de tu fork, así que primero necesitas sincronizar tu fork. Así se hace:

Línea de comandos solo con git

Los pasos 1 y 2 también se pueden hacer con el GitHub CLI o la interfaz web de GitHub.

  1. Añade upstream si todavía no lo has hecho
    git remote add upstream https://github.com/nikblanchet/docimp.git
  2. Descarga y actualiza tu main local
    git fetch upstream
    git checkout main
    git merge upstream/main
  3. Haz rebase de tu feature branch
    git checkout fix/gitignore-e2e-artifacts
    git rebase main
  4. Haz push del branch actualizado (hay que forzar el push después del rebase)
    git push --force-with-lease

Híbrido con GitHub CLI

Si tienes el GitHub CLI instalado, puedes reemplazar los pasos 1 y 2 con estos:

  1. Sincroniza tu fork con upstream
    gh repo sync pvpeuve/docimp -b main
  2. Descarga los cambios a tu main local
    git checkout main
    git pull

Híbrido con interfaz web de GitHub

Alternativamente, puedes reemplazar los pasos 1 y 2 con estos:

  1. En la página del branch main de tu fork, haz clic en Sync fork.

    Botón Sync fork resaltado en la página del repositorio

  2. Descarga los cambios a tu main local
    git checkout main
    git pull

Cómo aceptar cambios sugeridos

¡GitHub ya tiene excelente documentación para esto!

pvpeuve and others added 2 commits November 26, 2025 22:43
Co-authored-by: Nik Blanchet <nikblanchet@users.noreply.github.com>
@nikblanchet nikblanchet merged commit e9b4c07 into nikblanchet:main Nov 27, 2025
5 checks passed
@pvpeuve pvpeuve deleted the fix/gitignore-e2e-artifacts branch November 27, 2025 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add E2E test artifacts to .gitignore

2 participants