Skip to content

doc: run license-builder - #54322

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
actions/license-builder
Closed

doc: run license-builder#54322
github-actions[bot] wants to merge 1 commit into
mainfrom
actions/license-builder

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

License is likely out of date. This is an automatically generated PR by the license-builder.yml GitHub Action, which runs license-builder.sh and submits a new PR or updates an existing PR.

@github-actionsgithub-actionsBot added the meta Issues and PRs related to the general management of the project. label Aug 12, 2024
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@targostargos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

richardlau
richardlau previously approved these changes Aug 12, 2024
@richardlaurichardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 12, 2024
@avivkelleravivkeller removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 12, 2024
@avivkeller

Copy link
Copy Markdown
Member

Hey, just wanted to point out that the license builder script accounted for the header, so if this lands, the license wouldn't match that of c-ares.

The script should be updated instead of landing this IMO.

@avivkelleravivkeller added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 12, 2024
@avivkeller

Copy link
Copy Markdown
Member

I suggest not landing this, and instead landing #54335

@richardlaurichardlau removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 12, 2024
@richardlau
richardlau dismissed their stale reviewAugust 12, 2024 15:12

Would prefer to land #54335

@targostargos closed this Aug 13, 2024
@targos
targos deleted the actions/license-builder branch August 13, 2024 10:58
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

metaIssues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@nodejs-github-bot@avivkeller@targos@richardlau@aymen94@marco-ippolito