Skip to content

Conversation

@marksilcox
Copy link
Contributor

Motivation

Copying change from apply-config-from-env.py to ensure new line included before appending

Verifying this change

  • Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): ( no)
  • The public API: ( no)
  • The schema: ( no)
  • The default values of configurations: ( no)
  • The wire protocol: ( no)
  • The rest endpoints: ( no)
  • The admin cli options: ( no)
  • Anything that affects deployment: ( no )

Documentation

Need to update docs?

  • no-need-doc
    (Please explain why)

@merlimat merlimat added this to the 2.11.0 milestone May 13, 2022
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label May 13, 2022
@Technoboy- Technoboy- merged commit 74b2c70 into apache:master May 14, 2022
@marksilcox marksilcox deleted the fix_apply_config_prefix branch August 31, 2022 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants