Skip to content

Add Apache 2.0 LICENSE file - #31

Merged
luisina-santos merged 1 commit into
mainfrom
will.bengtson/add-apache-2-license
Oct 5, 2026
Merged

luisina-santos merged 1 commit into
mainfrom
will.bengtson/add-apache-2-license

Conversation

@willbengtson-c1rew

Copy link
Copy Markdown
Contributor

This repository is public but has no license file, so it is not open source under default copyright.

Adds the standard Apache License 2.0 text as LICENSE, matching baton-sdk and the other baton connectors.

No code changes.

🤖 Generated with Claude Code

The repository is public but had no license file. Add the Apache License 2.0
text, matching the other baton connectors and baton-sdk.

Co-authored-by: c1-squire-dev[bot] <c1-squire-dev[bot]@users.noreply.github.com>
@github-actions

github-actions Bot commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor
Superseded — see the current review report for commit e22d3f9ecee3

Connector PR Review: Add Apache 2.0 LICENSE file

Blocking Issues: 0 | Suggestions: 0 | Threads Resolved: 0
Criteria: Criteria status: loaded .claude/skills/ci-review.md from trusted base 0f0d7cdb4d3b.
Review mode: full
View review run

Review Summary

This PR adds one new file, LICENSE, containing the standard Apache License 2.0 text (202 lines). It runs from the header through "END OF TERMS AND CONDITIONS" and includes the usual APPENDIX boilerplate, with the [yyyy] / [name of copyright owner] placeholders left as-is, the same way upstream distributes it. That matches the PR description: there are no Go code, go.mod/go.sum, config or docs changes. I scanned the full PR diff for security and correctness issues and found none. The trusted repo-local criteria loaded and I applied them. Their sections (log levels, error wrapping, spans, JSON types, breaking-change gate, provisioning, ID stability, sync data loss) only cover connector source, so none of them apply to a license-only change. The connector mixin checks don't apply for the same reason. I found no new issues, and there were no prior findings to audit.

Security Issues

None found.

Correctness Issues

None found.

Suggestions

None.

@github-actions

github-actions Bot commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

Connector PR Review: Add Apache 2.0 LICENSE file

Blocking Issues: 0 | Suggestions: 0 | Threads Resolved: 0
Criteria: Criteria status: loaded .claude/skills/ci-review.md from trusted base 0f0d7cdb4d3b.
Review mode: full
View review run

Review Summary

This PR adds one new file, LICENSE, containing the standard Apache License 2.0 text (202 lines). It runs from the header through "END OF TERMS AND CONDITIONS" and includes the usual APPENDIX boilerplate, with the [yyyy] / [name of copyright owner] placeholders left as-is, the same way upstream distributes it. That matches the PR description: there are no Go code, go.mod/go.sum, config or docs changes. I scanned the full PR diff for security and correctness issues and found none. The trusted repo-local criteria loaded and I applied them. Their sections (log levels, error wrapping, spans, JSON types, breaking-change gate, provisioning, ID stability, sync data loss) only cover connector source, so none of them apply to a license-only change. The connector mixin checks don't apply for the same reason. I found no new issues, and there were no prior findings to audit.

Security Issues

None found.

Correctness Issues

None found.

Suggestions

None.


Reviewed commit: e22d3f9ecee3

@github-actions github-actions Bot 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.

No blocking issues found — see the full review report

@luisina-santos
luisina-santos merged commit 0c5ce25 into main Oct 5, 2026
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