Skip to content

chore: update Sentry license attribution#548

Merged
marandaneto merged 1 commit into
mainfrom
chore/update-sentry-license-attribution
Jun 3, 2026
Merged

chore: update Sentry license attribution#548
marandaneto merged 1 commit into
mainfrom
chore/update-sentry-license-attribution

Conversation

@marandaneto

Copy link
Copy Markdown
Member

💡 Motivation and Context

Clarifies attribution for code derived from getsentry/sentry-android-gradle-plugin and aligns the repository license notice with the file headers.

💚 How did you test it?

Not run (license/header-only changes).

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • I updated the docs if needed.
  • No breaking change or entry added to the changelog.

If releasing new changes

  • Ran pnpm changeset to generate a changeset file

@marandaneto marandaneto requested a review from a team as a code owner June 3, 2026 16:31
@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor
⚠️ `posthog-android-gradle-plugin` is modified but this PR has no changeset

This is informational — the PR is not blocked. Click the triangle above to collapse, or push a fix and this comment will auto-delete.

Modified in this PR but no changeset added:

  • posthog-android-gradle-plugin

If this change should ship, run pnpm changeset and select a bump level.
If it isn't user-facing (refactor with no behavior change, internal tooling, generated files), no action needed.

@greptile-apps

greptile-apps Bot commented Jun 3, 2026

Copy link
Copy Markdown
Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
LICENSE.md:48
The final line of the Sentry license block is now missing a trailing newline — the diff shows `\ No newline at end of file`. Adding a blank line (or just ensuring the file ends with `
`) after `SOFTWARE.` would restore the expected POSIX text-file format and prevent tools that check for missing newlines from flagging the file.

Reviews (1): Last reviewed commit: "chore: update Sentry license attribution" | Re-trigger Greptile

Comment thread LICENSE.md
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.
SOFTWARE. No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 The final line of the Sentry license block is now missing a trailing newline — the diff shows \ No newline at end of file. Adding a blank line (or just ensuring the file ends with ) after SOFTWARE. would restore the expected POSIX text-file format and prevent tools that check for missing newlines from flagging the file.

Prompt To Fix With AI
This is a comment left during a code review.
Path: LICENSE.md
Line: 48

Comment:
The final line of the Sentry license block is now missing a trailing newline — the diff shows `\ No newline at end of file`. Adding a blank line (or just ensuring the file ends with `
`) after `SOFTWARE.` would restore the expected POSIX text-file format and prevent tools that check for missing newlines from flagging the file.

How can I resolve this? If you propose a fix, please make it concise.

@marandaneto marandaneto merged commit afda3c8 into main Jun 3, 2026
14 checks passed
@marandaneto marandaneto deleted the chore/update-sentry-license-attribution branch June 3, 2026 19:52
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