Skip to content

remove unused runsettings#7305

Merged
afbrock merged 1 commit intov2from
adam/remove-runsetting
Apr 29, 2026
Merged

remove unused runsettings#7305
afbrock merged 1 commit intov2from
adam/remove-runsetting

Conversation

@afbrock
Copy link
Copy Markdown
Contributor

@afbrock afbrock commented Apr 28, 2026

Why are the changes needed?

Two fields were added to the settings idl when the values should stay in other services.

What changes were proposed in this pull request?

This change simply removes some extra fields. Nothing is using this yet.

How was this patch tested?

Labels

Please add one or more of the following labels to categorize your PR:

  • added: For new features.
  • changed: For changes in existing functionality.
  • deprecated: For soon-to-be-removed features.
  • removed: For features being removed.
  • fixed: For any bug fixed.
  • security: In case of vulnerabilities

This is important to improve the readability of release notes.

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Stack

If you do use git town to manage PR Stacks, the stack relevant to this PR
will show below. Otherwise, you can ignore this section.

Docs link

Signed-off-by: Adam Brock <adam@union.ai>
@afbrock afbrock force-pushed the adam/remove-runsetting branch from 3e05305 to 362ed5f Compare April 28, 2026 20:29
@afbrock afbrock merged commit 5f51ef7 into v2 Apr 29, 2026
21 checks passed
@afbrock afbrock deleted the adam/remove-runsetting branch April 29, 2026 21:39
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