Skip to content

Update pyenv#1591

Merged
filiptronicek merged 1 commit into
mainfrom
ft/bump-pyenv
May 6, 2025
Merged

Update pyenv#1591
filiptronicek merged 1 commit into
mainfrom
ft/bump-pyenv

Conversation

@filiptronicek
Copy link
Copy Markdown
Member

Description

Make the CI pass again (hopefully enough :P)

@filiptronicek filiptronicek requested review from a team as code owners May 6, 2025 07:53
@filiptronicek filiptronicek enabled auto-merge (squash) May 6, 2025 08:20
Copy link
Copy Markdown
Contributor

@mustard-mh mustard-mh left a comment

Choose a reason for hiding this comment

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

I saw error message on main build

 step 01      testing "it should have a functioning java 17 installed"command=[sdk default java 17.0.14.fx-zulu && java -version && mvn -v] 
              failed                                      message="assertion failed: status == 0" result="&test.RunResult{Stdout: []byte(""), Stderr: []byte("java 17.0.14.fx-zulu is not installed on your system\n"), StatusCode: 1}" 

That it's failed because the java unit tests, not sure why upgrade py could fix it but it's not harmful 🚢

@filiptronicek
Copy link
Copy Markdown
Member Author

@mustard-mh I see, that's another thing we need to fix. The error I was seeing in builds was a missing python version, which is what this PR is fixing.

@filiptronicek filiptronicek merged commit 699d195 into main May 6, 2025
4 checks passed
@filiptronicek filiptronicek deleted the ft/bump-pyenv branch May 6, 2025 09:50
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.

2 participants