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

[test]Update NOTICE.txt #39

Merged
merged 1 commit into from
Oct 24, 2024
Merged

[test]Update NOTICE.txt #39

merged 1 commit into from
Oct 24, 2024

Conversation

CalvinKirs
Copy link
Owner

Proposed changes

Issue Number: close #xxx

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

Copy link

This issue is available for anyone to work on. Make sure to reference this issue in your pull request. ✨ Thank you for your contribution! ✨

Copy link

Thank you for your contribution to Apache Doris.

Don't know what should be done next? See How to process your PR

Copy link

sh-checker report

To get the full details, please check in the job output.

shellcheck errors
'shellcheck ' found no issues.

shfmt errors

'shfmt ' returned error 1 finding the following formatting issues:

----------
--- env.sh.orig
+++ env.sh
@@ -163,7 +163,6 @@
     fi
     export LLVM_PROFDATA
 
-
     if [[ -z "${ENABLE_PCH}" ]]; then
         ENABLE_PCH='ON'
     fi
--- run-fe-ut.sh.orig
+++ run-fe-ut.sh
@@ -16,8 +16,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-
-
 set -eo pipefail
 
 ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")" &>/dev/null && pwd)"
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt  -w filename


@CalvinKirs CalvinKirs merged commit 7fba523 into master Oct 24, 2024
19 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant