Skip to content

docs: add Author (OneSignal) to README#2610

Closed
abdulraqeeb33 wants to merge 1 commit intomainfrom
cursor/readme-author-onesignal-87ca
Closed

docs: add Author (OneSignal) to README#2610
abdulraqeeb33 wants to merge 1 commit intomainfrom
cursor/readme-author-onesignal-87ca

Conversation

@abdulraqeeb33
Copy link
Copy Markdown
Contributor

Adds an Author section to README.md identifying OneSignal as the author, placed after the badges and before the migration advisory so it is easy to find.

Open in Web Open in Cursor 

Co-authored-by: abdulraqeeb33 <abdulraqeeb33@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 10, 2026 16:02
Copy link
Copy Markdown

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

Adds an Author section to the top of README.md to explicitly attribute the project to OneSignal.

Changes:

  • Inserted an #### Author heading after the badges.
  • Added OneSignal as the author attribution.

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

Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

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

LGTM — simple docs-only addition.

Extended reasoning...

Overview

This PR adds a 3-line "Author" section to README.md, placing "OneSignal" as a heading after the badges and before the migration advisory. No code, configuration, or logic is changed.

Security Risks

None. This is a documentation-only change with no executable content.

Level of Scrutiny

Minimal scrutiny required. The change is trivially small, mechanically correct, and follows the existing README structure.

Other Factors

No bugs were found by the automated system. No prior reviews exist on this PR. The repo already has a recent commit (5110ae8) with essentially the same intent, so this is consistent with project direction.

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