Skip to content

Update NOTICE to add license declaration for sphinx - #255

Merged
Radhika Gupta (rads-1996) merged 3 commits into
microsoft:mainfrom
rads-1996:update-notice
Sep 1, 2026
Merged

Update NOTICE to add license declaration for sphinx#255
Radhika Gupta (rads-1996) merged 3 commits into
microsoft:mainfrom
rads-1996:update-notice

Conversation

@rads-1996

Copy link
Copy Markdown
Member

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.

🟢 Approval recommended

The change is limited to NOTICE text updates and appears internally consistent and complete for the stated attribution purpose.

Pull request overview

Updates the repository’s third-party notices by adding an explicit Sphinx license declaration to NOTICE.md, aligning the NOTICE contents with dependency attribution obligations.

Changes:

  • Fixes formatting/indentation of the concluding NOTICE sentence so it’s a normal paragraph line.
  • Adds the Sphinx BSD 2-clause license notice text to NOTICE.md.
File summaries
File Description
NOTICE.md Adds Sphinx license notice text and normalizes NOTICE formatting for third-party attribution.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Performance comparison

Threshold: regressions >15.0% on gating scenarios fail the build. Higher ops/s is better; positive Δ means the PR is slower.

Scenario Gating Baseline (ops/s) Candidate (ops/s) Δ % Status
azure_monitor_log yes 19,803.9 20,006.8 -1.01%
azure_monitor_span yes 159,438.8 152,392.6 +4.62%
otel_log no 23,024.5 23,336.7 -1.34%
otel_span no 41,008.8 40,739.8 +0.66%

@rads-1996 Radhika Gupta (rads-1996) added the skip-changelog Not required for changes to test files, github actions, etc. label Sep 1, 2026
@rads-1996
Radhika Gupta (rads-1996) merged commit 455571c into microsoft:main Sep 1, 2026
12 checks passed
@rads-1996
Radhika Gupta (rads-1996) deleted the update-notice branch September 1, 2026 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Not required for changes to test files, github actions, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants