Skip to content

Use Apache 2.0 license boilerplate in NOTICE file - #118

Merged
bobbui merged 2 commits into
bobbui:masterfrom
Exagone313:update-notice-file
Aug 16, 2026
Merged

Use Apache 2.0 license boilerplate in NOTICE file#118
bobbui merged 2 commits 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 the copyright attribution and Apache License 2.0 licensing notice.

@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

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 33f6ac42-6b3b-4a2c-866e-2ab30c36c524

📥 Commits

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

📒 Files selected for processing (1)
  • NOTICE

📝 Walkthrough

Walkthrough

The NOTICE file replaces the previous attribution with Bob Bui and json-logging contributors. It adds a separate copyright line and the Apache License 2.0 terms and disclaimer.

Changes

NOTICE attribution update

Layer / File(s) Summary
NOTICE content replacement
NOTICE
Replaces the previous attribution with Bob Bui and json-logging contributors. Adds Copyright 2017 Bob Bui and Apache License 2.0 text.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: rednikisfun

🚥 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.

@bobbui
bobbui merged commit 8591029 into bobbui:master Aug 16, 2026
10 of 11 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.

2 participants