Skip to content

fix: Update LICENSE files to include additional dependencies and thei…#6418

Merged
Aias00 merged 1 commit into
masterfrom
release/fix_license
Jul 8, 2026
Merged

fix: Update LICENSE files to include additional dependencies and thei…#6418
Aias00 merged 1 commit into
masterfrom
release/fix_license

Conversation

@Aias00

@Aias00 Aias00 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

…r licenses

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Copilot AI review requested due to automatic review settings July 8, 2026 00:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the distribution LICENSE manifests for ShenYu bootstrap and admin artifacts to reflect additional third-party dependencies and their licensing, supporting release/compliance documentation for packaged binaries.

Changes:

  • Adds additional dependency/license entries (e.g., fastjson, error_prone_annotations, jakarta.mail-api, oshi-core 6.10.0).
  • Introduces a new “LGPL licenses” section listing jna / jna-platform.
  • Extends MIT/EPL dependency lists for the admin and bootstrap distributions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
shenyu-dist/shenyu-bootstrap-dist/src/main/release-docs/LICENSE Adds new dependency entries and an LGPL section for bootstrap distribution licensing docs.
shenyu-dist/shenyu-admin-dist/src/main/release-docs/LICENSE Adds new dependency entries and an LGPL section for admin distribution licensing docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +656 to +657
fastjson 1.2.76: https://github.com/alibaba/fastjson, Apache 2
error_prone_annotations 2.5.1: http://www.apache.org/licenses/LICENSE-2.0.txt, Apache 2.0
mcp-json 0.17.0: https://github.com/modelcontextprotocol/java-sdk, MIT
mcp-json-jackson2 0.17.0: https://github.com/modelcontextprotocol/java-sdk, MIT
mcp-spring-webflux 0.17.0: https://github.com/modelcontextprotocol/java-sdk, MIT
oshi-core 6.10.0: https://github.com/oshi/oshi, MIT
Comment on lines +840 to +841
jna 5.18.1: https://mvnrepository.com/artifact/net.java.dev.jna/jna, LGPL-2.1-or-later
jna-platform 5.18.1: https://mvnrepository.com/artifact/net.java.dev.jna/jna-platform, LGPL-2.1-or-later
Comment on lines 583 to +584
oshi-core 6.7.0: https://github.com/oshi/oshi, MIT
oshi-core 6.10.0: https://github.com/oshi/oshi, MIT
Comment on lines +754 to +755
jna 5.18.1: https://mvnrepository.com/artifact/net.java.dev.jna/jna, LGPL-2.1-or-later
jna-platform 5.18.1: https://mvnrepository.com/artifact/net.java.dev.jna/jna-platform, LGPL-2.1-or-later
@Aias00
Aias00 merged commit bc4f20d into master Jul 8, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants