Skip to content

Fix Build Issues#214

Merged
pauloborba merged 2 commits into
spgroup:masterfrom
leonardoAnjos16:chore/build-issues
Oct 28, 2025
Merged

Fix Build Issues#214
pauloborba merged 2 commits into
spgroup:masterfrom
leonardoAnjos16:chore/build-issues

Conversation

@leonardoAnjos16
Copy link
Copy Markdown
Contributor

Meus ultimos PRs causaram dois erros no CI:

  1. Atualizei a versão do Java no build.gradle, mas não tinha atualizado no Dockerfile
  2. Ao introduzir a opção de --random-seed, os commits coletados passaram a ser processador de forma aleatória (com seed padrão 1). Alterei os testes com a nova saída esperada (reordenando os commits na ordem em que sao processados quando se usa seed 1)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes two CI build issues: updating the Java version in the Dockerfile to match the build.gradle configuration, and adjusting test data to reflect the new default random seed behavior for commit processing order.

  • Updated Java version from 8 to 17 in Dockerfile to match build.gradle
  • Modified test CSV files to reflect the reordered commit processing when using default random seed 1

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
Dockerfile Updated Java runtime from version 8 to 17 in both build and runtime stages
src/test/integration/fileTest/compare/statistics/result.csv Reordered commit entries to match processing order with default random seed
src/test/integration/fileTest/compare/data/result.csv Reordered commit entries and renumbered lines to match new processing sequence

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread Dockerfile
@pauloborba pauloborba merged commit c453657 into spgroup:master Oct 28, 2025
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.

4 participants