Skip to content

Use Apache 2.0 license boilerplate in NOTICE file#118

Open
Exagone313 wants to merge 1 commit into
bobbui:masterfrom
Exagone313:update-notice-file
Open

Use Apache 2.0 license boilerplate in NOTICE file#118
Exagone313 wants to merge 1 commit into
bobbui:masterfrom
Exagone313:update-notice-file

Conversation

@Exagone313

@Exagone313 Exagone313 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Refs #93

Summary by CodeRabbit

  • Chores
    • Updated copyright attribution and Apache License 2.0 licensing notice in project documentation.

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

Grey Divider

Sorry, something went wrong

We weren't able to complete the code review on our side. Please try again

Grey Divider

ⓘ You are approaching your monthly quota for Qodo. Upgrade your plan

Qodo Logo

@Exagone313

Copy link
Copy Markdown
Contributor Author

@bobbui let me know if you prefer to use your username rather than your full name

@llamapreview llamapreview Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Auto Pull Request Review from LlamaPReview

Review Status: Automated Review Skipped

Dear contributor,

Thank you for your Pull Request. LlamaPReview has analyzed your changes and determined that this PR does not require an automated code review.

Analysis Result:

PR contains only modifications to the NOTICE file, updating the copyright notice and license boilerplate text. This is a purely legal/documentation change with no impact on code logic, functionality, or dependencies.

We're continuously improving our PR analysis capabilities. Have thoughts on when and how LlamaPReview should perform automated reviews? Share your insights in our GitHub Discussions.

Best regards,
LlamaPReview Team

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the NOTICE file to correct the copyright attribution to Bui Nguyen Thang and json-logging contributors, and appends the Apache License, Version 2.0 terms. I have no feedback to provide as there are no code changes or review comments.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e66aab7c-cd45-4e4d-a0b0-091616fd09d3

📥 Commits

Reviewing files that changed from the base of the PR and between ac68fce and 4332d5c.

📒 Files selected for processing (1)
  • NOTICE

📝 Walkthrough

Walkthrough

The NOTICE file is fully replaced: the previous copyright attributing "json-logging-python" and "Bob T." with a product inclusion notice is removed, and the new content names "Bui Nguyen Thang and json-logging contributors" as copyright holders with the standard Apache License 2.0 notice and URL.

Changes

NOTICE File Attribution Update

Layer / File(s) Summary
NOTICE file content replacement
NOTICE
Entire file body replaced: old attribution to "json-logging-python"/"Bob T." and product inclusion notice removed; new copyright line for "Bui Nguyen Thang and json-logging contributors" and full Apache License 2.0 notice text added.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • bobbui/json-logging-python#117: Directly related — that PR introduced the NOTICE file for Apache 2.0 compliance, and this PR updates its attribution content.

Poem

🐰 A NOTICE once named the wrong folk, you see,
The rabbit hopped in and fixed it with glee!
Now Bui Nguyen Thang gets the credit due,
Apache 2.0 shines bright, fresh and new.
With a twitch of my nose, the old text is gone —
Hop hop, little repo, carry on! 🌟

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the primary change: updating the NOTICE file to use Apache 2.0 license boilerplate, which is reflected in the file modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

Stopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a @coderabbit review after the pipeline has finished.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant