Adapt the license checker to other types of files #1033

huitseeker · 2022-03-22T18:43:54Z

We have PRs adding license headers to all the :

move Add license headers to all move files #1022
TS Add license headers for all typescript files #1028
files in the repo. But on its own, this will quickly bit-rot. We should adapt the script that checks a new file has the correct headers, and make sure it is triggered in CI, the way it is for rust files.

…edExecutor (MystenLabs#1033) Before we make a decision on whether and how to remove the BoundedExecutor, it seems useful to increase the max concurrency limit significantly so we are not running into it as often. Blocking on BoundedExecutor is potentially one source of deadlock we have observed.

huitseeker assigned 666lcz Mar 22, 2022

huitseeker changed the title ~~Adapt the license checker to~~ Adapt the license checker to other types of files Mar 22, 2022

666lcz linked a pull request Mar 23, 2022 that will close this issue

Add license check for move, typescript, javascript files #1051

Merged

666lcz closed this as completed in #1051 Mar 24, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adapt the license checker to other types of files #1033

Adapt the license checker to other types of files #1033

huitseeker commented Mar 22, 2022 •

edited

Loading

Adapt the license checker to other types of files #1033

Adapt the license checker to other types of files #1033

Comments

huitseeker commented Mar 22, 2022 • edited Loading

huitseeker commented Mar 22, 2022 •

edited

Loading