Skip to content

Conversation

@SouradeepSaha
Copy link
Contributor

Description

Trim unnecessary whitespace at the beginning and end of files in the unittest/ directory. Only addresses a subset of files from over 5500+ files currently having unnecessary whitespace. Note that all code changes are non-functional.

Release Notes

N/A

How can this PR be tested?

All tests are confirmed to pass by running the command ./mtr --suite=main.

Basing the PR against the correct MariaDB version

  • This is a bug fix and the PR is based against the earliest maintained branch in which the bug can be reproduced.

PR quality check

  • I checked the CODING_STANDARDS.md file and my PR conforms to this where appropriate.
  • For any trivial modifications to the PR, I am ok with the reviewer making the changes themselves.

All new code of the whole pull request, including one or several files that are either new files or modified ones, are contributed under the BSD-new license. I am contributing on behalf of my employer Amazon Web Services, Inc.

Cleanup unnecessary whitespace at the end of lines and end of files
in the unittest/ directory. Note that all code changes are
non-functional.

All new code of the whole pull request, including one or several files
that are either new files or modified ones, are contributed under the
BSD-new license. I am contributing on behalf of my employer Amazon Web
Services, Inc.
@CLAassistant
Copy link

CLAassistant commented Jun 29, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ grooverdan
❌ SouradeepSaha
You have signed the CLA already but the status is still pending? Let us recheck it.

@grooverdan grooverdan enabled auto-merge (rebase) July 17, 2024 04:05
@grooverdan grooverdan merged commit 4dde925 into MariaDB:11.6 Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants