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

license-header-check failed #814

Closed
evenyag opened this issue Jan 3, 2023 · 2 comments · Fixed by #815 or GreptimeTeam/promql-parser#19
Closed

license-header-check failed #814

evenyag opened this issue Jan 3, 2023 · 2 comments · Fixed by #815 or GreptimeTeam/promql-parser#19
Labels
C-bug Category Bugs

Comments

@evenyag
Copy link
Contributor

evenyag commented Jan 3, 2023

What type of bug is this?

Other

What subsystems are affected?

Other

What happened?

The license-header-check action failed

What operating system did you use?

Ubuntu

Relevant log output and stack trace

/usr/bin/docker run --name a1b490347437687e676a9187b6ac3_5e0aca --label 290506 --workdir /github/workspace --rm -e "INPUT_LOG" -e "INPUT_CONFIG" -e "INPUT_TOKEN" -e "INPUT_MODE" -e "GITHUB_TOKEN" -e "HOME" -e "GITHUB_JOB" -e "GITHUB_REF" -e "GITHUB_SHA" -e "GITHUB_REPOSITORY" -e "GITHUB_REPOSITORY_OWNER" -e "GITHUB_RUN_ID" -e "GITHUB_RUN_NUMBER" -e "GITHUB_RETENTION_DAYS" -e "GITHUB_RUN_ATTEMPT" -e "GITHUB_ACTOR" -e "GITHUB_TRIGGERING_ACTOR" -e "GITHUB_WORKFLOW" -e "GITHUB_HEAD_REF" -e "GITHUB_BASE_REF" -e "GITHUB_EVENT_NAME" -e "GITHUB_SERVER_URL" -e "GITHUB_API_URL" -e "GITHUB_GRAPHQL_URL" -e "GITHUB_REF_NAME" -e "GITHUB_REF_PROTECTED" -e "GITHUB_REF_TYPE" -e "GITHUB_WORKSPACE" -e "GITHUB_ACTION" -e "GITHUB_EVENT_PATH" -e "GITHUB_ACTION_REPOSITORY" -e "GITHUB_ACTION_REF" -e "GITHUB_PATH" -e "GITHUB_ENV" -e "GITHUB_STEP_SUMMARY" -e "GITHUB_STATE" -e "GITHUB_OUTPUT" -e "RUNNER_OS" -e "RUNNER_ARCH" -e "RUNNER_NAME" -e "RUNNER_TOOL_CACHE" -e "RUNNER_TEMP" -e "RUNNER_WORKSPACE" -e "ACTIONS_RUNTIME_URL" -e "ACTIONS_RUNTIME_TOKEN" -e "ACTIONS_CACHE_URL" -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/greptimedb/greptimedb":"/github/workspace" 290506:539a1b490347437687e676a9187b6ac3  "-v" "info" "-c" ".licenserc.yaml" "header" "check"
INFO Loading configuration from file: .licenserc.yaml 
INFO Totally checked 646 files, valid: 0, invalid: 540, ignored: 106, fixed: 0 
WARNING Failed to create review comment, fallback to a plain comment: POST https://api.github.com/repos/GreptimeTeam/greptimedb/pulls/801/reviews: 403 Resource not accessible by integration [] 
WARNING Failed to create review comments POST https://api.github.com/repos/GreptimeTeam/greptimedb/pulls/801/reviews: 403 Resource not accessible by integration [] 
ERROR the following files don't have a valid license header: 
benchmarks/src/bin/nyc-taxi.rs
src/api/build.rs
src/api/src/error.rs
src/api/src/helper.rs
src/api/src/lib.rs
src/api/src/prometheus.rs
src/api/src/result.rs

How can we reproduce the bug?

Run the ci
@evenyag evenyag added the C-bug Category Bugs label Jan 3, 2023
@evenyag
Copy link
Contributor Author

evenyag commented Jan 3, 2023

@v0y4g3r
Copy link
Contributor

v0y4g3r commented Jan 3, 2023

Looks like we need to update year and indentation in license header. Maybe try

find  .  -type f  -iname "*.rs" | \
    xargs -L1 -I{} sed -i 's/\/\/ Copyright 2022 Greptime Team/\/\/ Copyright 2023 Greptime Team/g' {}

and

find  .  -type f  -iname "*.rs" | \
    xargs -L1 -I{} sed -i 's/\/\/ http:\/\/www.apache.org\/licenses\/LICENSE-2.0/\/\/     http:\/\/www.apache.org\/licenses\/LICENSE-2.0/g' {}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category Bugs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants