Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Fix test runs #3782

Merged
merged 1 commit into from
Sep 23, 2023
Merged

Fix test runs #3782

merged 1 commit into from
Sep 23, 2023

Conversation

v1r3n
Copy link
Contributor

@v1r3n v1r3n commented Sep 23, 2023

  1. Disable WorkflowCreationTest since it brings up a server and with JDK17, the test will not pass until a JDK17+ build is published.
  2. Clean up the ci.yaml

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • [x ] Other (please describe):
    Fixes to the tests to support JDK17 move.

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

Describe the new behavior from this PR, and why it's needed
Issue #

Alternatives considered

Describe alternative implementation you have considered

1. Disable WorkflowCreationTest since it brings up a server and with JDK17, the test will not pass until a JDK17+ build is published.
2. Clean up the ci.yaml
@v1r3n v1r3n merged commit 1c899db into Netflix:main Sep 23, 2023
2 checks passed
@v1r3n v1r3n deleted the update_test_runs branch September 23, 2023 18:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant