-
Notifications
You must be signed in to change notification settings - Fork 217
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
chore(workflows): add skywalking-eyes
license header check
#2867
chore(workflows): add skywalking-eyes
license header check
#2867
Conversation
f16c204
to
7c41cfb
Compare
header: | ||
license: | ||
spdx-id: Apache-2.0 | ||
copyright-owner: contributors to the oneDAL project |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can it also check that older Intel copyright owner that still exist?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed this field so it won't be confusing, as it's not used for check.
7c41cfb
to
6ceea9b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would assume that overall check works and it would catch copyright that is not compliant
.github/.licenserc.yaml
Outdated
# Clang-format configs | ||
- '.clang-format' | ||
# All .md files | ||
- '**/*.md' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do have md files with copyrights - should we keep them for check? How many failures?
Might be as a next pr discussion on exclusion - fine to merge this one as is
f785071
to
e102d97
Compare
e102d97
to
b0b12d4
Compare
34e80b0
to
34afa43
Compare
eb9e77a
to
8b66580
Compare
/azp run |
304c789
to
703bee6
Compare
@Mergifyio rebase |
✅ Branch has been successfully rebased |
45c7cbc
to
555980c
Compare
ca894bd
to
5d22a83
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This review mirrors this one: uxlfoundation/scikit-learn-intelex#2215 (review)
permissions: | ||
contents: read |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
permissions: | |
contents: read | |
permissions: read-all |
permissions: | ||
issues: write | ||
pull-requests: write |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same question as in uxlfoundation/scikit-learn-intelex#2215
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The check can leave comments on the pull request if it finds issues. For this, write permissions need to be granted for these scopes.
5d22a83
to
55ca631
Compare
55ca631
to
dd4a916
Compare
Description
Added
skywalking-eyes
license header check:.github/workflows/skywalking-eyes.yml
workflow (actually, the check itself).github/.licenserc.yaml
config for checkerAdditionally:
Copyrights in some files were updated:
Added license header to the following
.md
files:.github/ISSUE_TEMPLATE/RFC.md
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/documentation.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/pull_request_template.md
CODE_OF_CONDUCT.md
Removed second year in copyright notice for the following files:
cpp/daal/src/externals/service_math_ref.h
cpp/oneapi/dal/algo/decision_forest/backend/gpu/train_kernel_hist_impl.hpp
cpp/oneapi/dal/algo/decision_forest/backend/gpu/train_kernel_hist_impl_dpc.cpp
cpp/oneapi/dal/algo/decision_forest/backend/gpu/train_service_kernels.hpp
cpp/oneapi/dal/algo/decision_forest/backend/gpu/train_service_kernels_dpc.cpp
Fixed framing for the following
.rst
files (indocs/source
):docs/source/404.rst
docs/source/api/algorithms/clustering/dbscan.rst
docs/source/api/algorithms/clustering/index.rst
docs/source/api/algorithms/clustering/kmeans-init.rst
docs/source/api/algorithms/clustering/kmeans.rst
docs/source/api/algorithms/covariance/covariance.rst
docs/source/api/algorithms/covariance/index.rst
...