Skip to content

[chore](FE)add license header check in fe's checkstyle#11076

Merged
924060929 merged 1 commit intoapache:masterfrom
morrySnow:license_check
Jul 22, 2022
Merged

[chore](FE)add license header check in fe's checkstyle#11076
924060929 merged 1 commit intoapache:masterfrom
morrySnow:license_check

Conversation

@morrySnow
Copy link
Contributor

Problem Summary:

Add license header check in fe's checkstyle

@github-actions github-actions bot added area/nereids area/planner Issues or PRs related to the query planner labels Jul 21, 2022
@morrySnow morrySnow requested a review from 924060929 July 21, 2022 10:52
@morrySnow
Copy link
Contributor Author

@924060929 PTAL

@wangshuo128
Copy link
Contributor

LGTM
I thought this already exists...

@wangshuo128
Copy link
Contributor

BTW, do we have any guidance to let developers get a license header template and config generating license header when creating new files automaticly in IntelliJ IDEA?

Copy link
Contributor

@924060929 924060929 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@924060929 924060929 merged commit d17c906 into apache:master Jul 22, 2022
@morrySnow morrySnow deleted the license_check branch July 25, 2022 12:10
miswujian pushed a commit to miswujian/doris that referenced this pull request Jul 28, 2022
whutpencil pushed a commit to whutpencil/incubator-doris that referenced this pull request Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/nereids area/planner Issues or PRs related to the query planner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants