Skip to content

Conversation

@iOvergaard
Copy link
Contributor

This pull request updates the third-party notices in the NOTICES.txt file to improve clarity and remove outdated entries. The most important changes include clarifying the scope of third-party dependencies and cleaning up the list of attributions.

Documentation improvements:

  • Clarified that third-party software may have dependencies not explicitly listed and updated the introductory notice for better accuracy.

Third-party attribution cleanup:

The following notices no longer apply with #19716:

  • Removed the attribution for the Glob library, which is no longer listed as a dependency.
  • Removed multiple Rollup-related attributions (Rollup, Rollup Plugins, Rollup-plugin-esbuild, and Rollup-plugin-import-css) from the notice file, indicating these are no longer relevant dependencies.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request updates the NOTICES.txt file by removing obsolete third-party software attributions and improving the clarity of the introductory notice. The changes reflect updates to the project's dependency structure following previous changes (referenced as #19716).

Key changes:

  • Enhanced the introductory notice to clarify the scope and limitations of third-party dependency listings
  • Removed attributions for build tools (Rollup and related plugins) that are no longer project dependencies
  • Removed attribution for the Glob library which is no longer used

@leekelleher leekelleher enabled auto-merge (squash) September 1, 2025 10:55
@iOvergaard iOvergaard disabled auto-merge September 1, 2025 11:10
@iOvergaard iOvergaard merged commit ae44f30 into main Sep 1, 2025
16 of 26 checks passed
@iOvergaard iOvergaard deleted the notices-remove-rollup branch September 1, 2025 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants