Skip to content

Conversation

@SarahFrench
Copy link
Member

While looking at backend-related code I found some comments that could be updated to reflect the idea of operations backends versus the old "enhanced" backends.

Fixing those in a separate PR to avoid noise.

Also, I found some duplicated comments that weren't accurate.

Target Release

1.12.x

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@SarahFrench SarahFrench added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Mar 19, 2025
@SarahFrench SarahFrench changed the title Sarah/fix comments Update comments about backend interfaces, enhanced vs operations, and fix copy-paste inaccuracies Mar 19, 2025
@SarahFrench SarahFrench marked this pull request as ready for review March 20, 2025 10:26
@SarahFrench SarahFrench requested a review from a team as a code owner March 20, 2025 10:26
@SarahFrench SarahFrench merged commit 2d6eda2 into main Mar 20, 2025
14 checks passed
@SarahFrench SarahFrench deleted the sarah/fix-comments branch March 20, 2025 16:52
rkoron007 pushed a commit that referenced this pull request Mar 27, 2025
… fix copy-paste inaccuracies (#36716)

* Fix copy-pasta

* Update some comments clarifying backend-related interfaces and "enhanced" versus "operations"

* Fix more comments that refer to types and interfaces that have moved into the backendrun package
@github-actions
Copy link
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants