Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

get rid of internal_config feature #1655

Open
milyin opened this issue Dec 10, 2024 · 0 comments
Open

get rid of internal_config feature #1655

milyin opened this issue Dec 10, 2024 · 0 comments
Labels
release Part of the next release

Comments

@milyin
Copy link
Contributor

milyin commented Dec 10, 2024

Describe the release item

The feature internal_config can be removed, it's functionality can be put under internal feature.

Though to achieve this goal one of these things should be done:

  • either the internal should not suppose unstable. E.g. test for liveliness which actively uses this write access to config should use internal feature, but not unstable. This is nesessary for run tests in stable mode #1654
  • or the tests should not use this direct access to config, using set methods instead
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Part of the next release
Projects
Status: No status
Development

No branches or pull requests

1 participant