Skip to content

Cleanup unused WV3 test code paths#9666

Merged
ShahabT merged 1 commit intotemporalio:mainfrom
ShahabT:wv3-test-cleanup
Mar 26, 2026
Merged

Cleanup unused WV3 test code paths#9666
ShahabT merged 1 commit intotemporalio:mainfrom
ShahabT:wv3-test-cleanup

Conversation

@ShahabT
Copy link
Copy Markdown
Contributor

@ShahabT ShahabT commented Mar 25, 2026

What changed?

  • Cleanup unused WV3 test code paths
  • Enable use revision numbers config by default

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

Potential risks

None

@ShahabT ShahabT requested review from a team as code owners March 25, 2026 21:52
@ShahabT ShahabT enabled auto-merge (squash) March 26, 2026 00:26
@ShahabT ShahabT merged commit 4d893b5 into temporalio:main Mar 26, 2026
69 of 71 checks passed
awln-temporal pushed a commit that referenced this pull request Mar 26, 2026
## What changed?
- Cleanup unused WV3 test code paths
- Enable use revision numbers config by default

## How did you test it?
- [ ] built
- [ ] run locally and tested manually
- [ ] covered by existing tests
- [ ] added new unit test(s)
- [ ] added new functional test(s)

## Potential risks
None
chaptersix pushed a commit to chaptersix/temporal that referenced this pull request Apr 2, 2026
## What changed?
- Cleanup unused WV3 test code paths
- Enable use revision numbers config by default

## How did you test it?
- [ ] built
- [ ] run locally and tested manually
- [ ] covered by existing tests
- [ ] added new unit test(s)
- [ ] added new functional test(s)

## Potential risks
None
chaptersix pushed a commit to chaptersix/temporal that referenced this pull request Apr 2, 2026
## What changed?
- Cleanup unused WV3 test code paths
- Enable use revision numbers config by default

## How did you test it?
- [ ] built
- [ ] run locally and tested manually
- [ ] covered by existing tests
- [ ] added new unit test(s)
- [ ] added new functional test(s)

## Potential risks
None
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