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

Change setting's deprecation message wording #80900

Closed

Conversation

pgomulka
Copy link
Contributor

@pgomulka pgomulka commented Nov 22, 2021

depending on whether a message is critical or warning a message should
indicate to check breaking changes documentation (critical level) or
deprecation changes documentation (warn level)

relates #79666

depending on whether a message is critical or warning a message shoudl
indicate to check breaking changes documentation (critical level) or
deprecation changes documentation (warn level)

relates elastic#79666
@danhermann danhermann added v7.16.1 and removed v7.16.0 labels Dec 3, 2021
@pgomulka
Copy link
Contributor Author

pgomulka commented Dec 9, 2021

ok to test

@pgomulka
Copy link
Contributor Author

pgomulka commented Dec 9, 2021

@elasticmachine update branch

@nik9000 nik9000 added :Core/Infra/Settings Settings infrastructure and APIs team-discuss and removed needs:triage Requires assignment of a team area label labels Aug 2, 2022
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Aug 2, 2022
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@grcevski
Copy link
Contributor

grcevski commented Aug 8, 2022

@pgomulka This looks like an older PR, but the change looks good to me. Have we decided if we wanted to proceed with this change? It appears that few other tests need to be adjusted.

@mosche
Copy link
Contributor

mosche commented Apr 18, 2024

As discussed, closing to be revisited for v9 (#107639)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.