Skip to content

fix: Scanner for invariants - followup#10596

Open
davidporter-id-au wants to merge 1 commit into
temporalio:mainfrom
davidporter-id-au:feature/adding-scanner-for-invariants-2
Open

fix: Scanner for invariants - followup#10596
davidporter-id-au wants to merge 1 commit into
temporalio:mainfrom
davidporter-id-au:feature/adding-scanner-for-invariants-2

Conversation

@davidporter-id-au
Copy link
Copy Markdown
Contributor

@davidporter-id-au davidporter-id-au commented Jun 8, 2026

What changed?

I let #10406 autoland and didn't respond to all feedback, addressing the remaining here.

  • Pulls all the dynamic config into a single struct
  • Switches Namespace list to used the local cached version
  • Removes the string indirection for registry
  • Converts some minor functions to be iterators

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

The same as the previous scanner PR

@davidporter-id-au davidporter-id-au force-pushed the feature/adding-scanner-for-invariants-2 branch from da6f2b6 to e43c05e Compare June 8, 2026 00:24
@davidporter-id-au davidporter-id-au marked this pull request as ready for review June 8, 2026 00:33
@davidporter-id-au davidporter-id-au requested review from a team as code owners June 8, 2026 00:33
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.

1 participant