Skip to content

🧹 Expose built in providers.#7076

Merged
preslavgerchev merged 1 commit intomainfrom
preslav/expose-builtin-providres
Mar 31, 2026
Merged

🧹 Expose built in providers.#7076
preslavgerchev merged 1 commit intomainfrom
preslav/expose-builtin-providres

Conversation

@preslavgerchev
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

@mondoo-code-review mondoo-code-review bot left a comment

Choose a reason for hiding this comment

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

Exported map exposes unexported type, limiting usability for external consumers.

@mondoo-code-review mondoo-code-review bot dismissed their stale review March 31, 2026 09:02

Superseded by new review

Copy link
Copy Markdown

@mondoo-code-review mondoo-code-review bot left a comment

Choose a reason for hiding this comment

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

Exposes built-in provider registry to external packages by exporting the type and variable.

Signed-off-by: Preslav <preslav@mondoo.com>
@preslavgerchev preslavgerchev force-pushed the preslav/expose-builtin-providres branch from b7e7367 to 031f5bf Compare March 31, 2026 09:04
Copy link
Copy Markdown

@mondoo-code-review mondoo-code-review bot left a comment

Choose a reason for hiding this comment

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

Exposes built-in provider names via a new public function with no issues.

@github-actions
Copy link
Copy Markdown
Contributor

Test Results

5 741 tests  ±0   5 737 ✅ ±0   2m 10s ⏱️ -1s
  425 suites ±0       4 💤 ±0 
   33 files   ±0       0 ❌ ±0 

Results for commit 031f5bf. ± Comparison against base commit 8b6941c.

@preslavgerchev preslavgerchev merged commit d6ffe35 into main Mar 31, 2026
22 checks passed
@preslavgerchev preslavgerchev deleted the preslav/expose-builtin-providres branch March 31, 2026 10:58
@github-actions github-actions bot locked and limited conversation to collaborators Mar 31, 2026
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