Skip to content

doc: run license-builder #61522

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.

Sign up for GitHub

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

Jump to bottom
Open
github-actions wants to merge 2 commits into
base: main
Choose a base branch
from
Open

doc: run license-builder #61522

github-actions wants to merge 2 commits into from
+203 −204

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jan 26, 2026

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-actions github-actions bot added the meta Issues and PRs related to the general management of the project. label Jan 26, 2026
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jan 26, 2026

Review requested:

  • @nodejs/tsc

@legendecas legendecas mentioned this pull request Jan 26, 2026
Merged
@legendecas legendecas marked this pull request as draft January 26, 2026 02:22
@legendecas
Copy link
Member

legendecas commented Jan 26, 2026

This should be blocked until #61523 lands.

nodejs-github-bot pushed a commit that referenced this pull request Jan 26, 2026
@legendecas
tools: add LIEF to license builder
d14b484 
PR-URL: #61523
Refs: #61522
Refs: #61167
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@legendecas legendecas marked this pull request as ready for review January 26, 2026 21:51
@aduh95 aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 26, 2026
joyeecheung
joyeecheung reviewed Jan 27, 2026
View reviewed changes
LICENSE
"""

- postject, located at test/fixtures/postject-copy and used as a basis for
src/node_sea_bin.cc, is licensed as follows:
Copy link
Member

@joyeecheung joyeecheung Jan 27, 2026

Choose a reason for hiding this comment

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

Not sure if it's fine to remove this mention - I suppose we can add a reference in the source code instead?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joyeecheung joyeecheung joyeecheung left review comments

@legendecas legendecas legendecas approved these changes

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. meta Issues and PRs related to the general management of the project.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@nodejs-github-bot @legendecas @joyeecheung @aduh95