Skip to content

(Fix): Bump runtime versions and fix tests #926

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 20 commits into from
Dec 5, 2024
Merged

(Fix): Bump runtime versions and fix tests #926

merged 20 commits into from
Dec 5, 2024

Conversation

TylerJang27
Copy link
Collaborator

@TylerJang27 TylerJang27 commented Nov 27, 2024 •
edited
Loading

Tis the season to be green. Fixes some tests that slipped while #920 was in the works:

  • Upgrade to jdk-23 (tested old snapshots)
  • Upgraded rust to 1.82.0
  • Upgraded php linters and tools to support php@8.4
  • Improved testing matchers

And readds the eslint fix from #920 🙃

@trunk-io Trunk.io
Copy link

trunk-io bot commented Nov 27, 2024 •
edited
Loading

⏱️ 20h 19m total CI duration on this PR
Job Cumulative Duration Recent Runs
Windows Linter Tests 7h 32m ⬜ ⬜ 🟩 ⬜ 🟩 (+7 more)
Linter Tests macOS 5h 🟥 ⬜ 🟥 🟥 🟩 (+7 more)
Linter Tests ubuntu-latest 4h 26m ⬜ 🟩 🟥 🟥 🟩 (+8 more)
Tool Tests (ubuntu-latest) 1h 25m 🟩 ⬜ 🟩 🟩 🟩 (+7 more)
Tool Tests (macOS) 59m 🟥 ⬜ 🟩 ⬜ 🟩 (+7 more)
Action Tests 18m 🟩 ⬜ 🟩 🟩 🟩 (+7 more)
CodeQL-Build 14m 🟩 🟩 🟩 🟩 🟩 (+7 more)
Repo Tests / Plugin Tests 11m 🟩 🟩 🟩 🟩 🟩 (+7 more)
Trunk Check runner [linux] 9m 🟩 🟩 🟩 🟩 🟩 (+7 more)
Windows Tool Tests 3m 🟩 🟩 🟩 🟩 
Detect changed files 1m 🟩 🟩 🟩 🟩 🟩 (+7 more)
Aggregate Test Results 32s 🟥 🟥 🟥 🟥 🟩 (+8 more)

settings ⋅ feedback ⋅ docs ⋅ learn more about trunk.io

@trunk-io Trunk.io
Copy link

trunk-io bot commented Nov 27, 2024 •
edited
Loading

Static Badge   Static Badge   Static Badge   Static Badge

Failed Test Failure Summary Logs
Testing linter detekt test basic_detekt Detekt linter failed with exit code 1, possibly due to issues with Java or Detekt configuration. Logs ↗︎
Testing linter detekt-gradle test CUSTOM Detekt Gradle linter failed to run due to an error in executing the detekt task. Logs ↗︎
Testing linter trunk-toolbox test do_not_land The linter test failed to find expected issues in the file. Logs ↗︎
Testing linter detekt-explicit test basic_explicit Detekt-explicit linter failed with exit code 1, unable to execute Java binary. Logs ↗︎

... and 8 more

Flaky Test Failure Summary Logs
Testing tool paratest tool Phive 0.15.1 deprecated some parameters, which caused an error while installing paratest. Logs ↗︎
Testing tool phpunit tool Phive 0.15.1 installation failed due to deprecated PHP warnings. Logs ↗︎

View Full Report ↗︎ ⋅ Docs

@TylerJang27 TylerJang27 marked this pull request as ready for review December 3, 2024 05:46
@TylerJang27 TylerJang27 requested a review from det December 3, 2024 05:47
@TylerJang27 TylerJang27 merged commit 40820bc into main Dec 5, 2024
15 checks passed
@TylerJang27 TylerJang27 deleted the tyler/fix-tests branch December 5, 2024 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@det det det approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TylerJang27 @det