Skip to content

[DOCS-2546] Add Apache 2.0 LICENSE files and remove conflicting license declarations #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Jun 3, 2025
Merged

[DOCS-2546] Add Apache 2.0 LICENSE files and remove conflicting license declarations #71

merged 3 commits into from
Jun 3, 2025

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Jun 2, 2025 •
edited by launchdarkly-upra bot
Loading

Add Apache 2.0 LICENSE files and remove conflicting license declarations

This PR adds consistent Apache 2.0 licensing across the entire observability-sdk repository by:

Changes Made

✅ Added LICENSE files

  • Repository root: Added LICENSE file with Apache 2.0 license
  • All 23 SDK directories: Added LICENSE files to each SDK under sdk/
    • @launchdarkly/observability/
    • @launchdarkly/session-replay/
    • highlight-apollo/, highlight-cloudflare/, highlight-dotnet/, highlight-dotnet4/
    • highlight-ex/, highlight-go/, highlight-hono/, highlight-java/
    • highlight-nest/, highlight-next/, highlight-node/, highlight-php/
    • highlight-pino/, highlight-py/, highlight-react/, highlight-remix/
    • highlight-ruby/, highlight-run/, highlight-rust/, highlight-sourcemap-uploader/
    • highlight-wordpress/

✅ Replaced existing licenses

  • highlight-go: Replaced MIT license with Apache 2.0
  • highlight-ruby: Replaced MIT license with Apache 2.0
  • highlight-php: Replaced full Apache license text with standardized version

✅ Removed conflicting license declarations

  • JavaScript/TypeScript packages: Removed "license": "Apache-2.0" from 12+ package.json files
  • Ruby: Removed spec.license = 'MIT' from highlight.gemspec
  • Java: Removed <licenses> blocks from 7 pom.xml files
  • Rust: Removed license = "Apache-2.0" from 6 Cargo.toml files
  • Python: Removed license = "Apache-2.0" from pyproject.toml
  • PHP: Removed license declaration from composer.json
  • Elixir: Removed licenses: ["Apache-2.0"] from mix.exs
  • .NET: Removed <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression> from .csproj

License Text

All LICENSE files use the standardized Apache 2.0 license with:

Copyright 2025 Catamorphic, Co.

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

    http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.

Verification

  • ✅ 53 files changed (25 LICENSE files created/updated, 28 configuration files updated)
  • ✅ No conflicting license declarations remain in SDK packages
  • ✅ Consistent Apache 2.0 licensing across entire repository

Link to Devin run: https://app.devin.ai/sessions/af316a82b2f14ff48bcc177afce58db0
Requested by: Vadim Korolik ([email protected])

Related Jira issue: DOCS-2546: add LICENSE file to launchdarkly/observability-SDK

@devin-ai-integration @Vadman97
Add Apache 2.0 LICENSE files and remove conflicting license declarations
717700e 
- Add LICENSE file at repository root with Apache 2.0 license
- Add LICENSE files to all 23 SDK directories under sdk/
- Replace existing MIT licenses in highlight-go and highlight-ruby with Apache 2.0
- Remove license declarations from package.json files across all SDKs
- Remove license blocks from Java pom.xml files
- Remove license declarations from Ruby gemspec, Rust Cargo.toml, .NET .csproj, Python pyproject.toml, PHP composer.json, and Elixir mix.exs files
- Ensure consistent Apache 2.0 licensing across entire observability-sdk repository

Co-Authored-By: Vadim Korolik <[email protected]>
@devin-ai-integration devin-ai-integration bot requested a review from a team as a code owner June 2, 2025 21:50
@devin-ai-integration
Copy link
Contributor Author

devin-ai-integration bot commented Jun 2, 2025

Original prompt from Vadim Korolik:



in observability-sdk , add a `LICENSE` file (at the top level and in each directory in `sdk/...` where an SDK is implemented) with the following contents, making sure no other license is defined for each SDK:

Copyright 2025 Catamorphic, Co.

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.


You only need to look in the following repo: launchdarkly/observability-sdk

@devin-ai-integration
Copy link
Contributor Author

devin-ai-integration bot commented Jun 2, 2025

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2025 •
edited
Loading

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
628 551 88% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 58e9d6f by action🐍

@devin-ai-integration @Vadman97
Fix .NET build by restoring required PackageLicenseExpression
3dfa420 
The PackageLicenseExpression is required for NuGet packaging when PackageRequireLicenseAcceptance is enabled. This works alongside the LICENSE file to provide complete licensing information for the .NET package.

Co-Authored-By: Vadim Korolik <[email protected]>
@Vadman97 Vadman97 changed the title Add Apache 2.0 LICENSE files and remove conflicting license declarations [DOCS-2546] Add Apache 2.0 LICENSE files and remove conflicting license declarations Jun 2, 2025
@devin-ai-integration @Vadman97
Restore Apache-2.0 license declarations in package definition files
58e9d6f 
- Restore 'license': 'Apache-2.0' in all JavaScript/TypeScript package.json files
- Restore <licenses> blocks in all Java pom.xml files
- Restore license = 'Apache-2.0' in all Rust Cargo.toml files
- Restore license declarations in Python, PHP, Elixir, and Ruby config files
- Keep all LICENSE files that were previously added
- Maintain both LICENSE files and package definition license declarations

Co-Authored-By: Vadim Korolik <[email protected]>
@Vadman97 Vadman97 merged commit 86026fa into main Jun 3, 2025
38 checks passed
@Vadman97 Vadman97 deleted the devin/1748900583-add-license-files branch June 3, 2025 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Vadman97 Vadman97 Vadman97 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Vadman97