Skip to content

chore: update LICENSE text - #2431

Merged
sdirix merged 1 commit into
masterfrom
adapt-license-text
Mar 26, 2025
Merged

chore: update LICENSE text#2431
sdirix merged 1 commit into
masterfrom
adapt-license-text

Conversation

@sdirix

@sdirix sdirix commented Mar 24, 2025

Copy link
Copy Markdown
Member

Updates the LICENSE text to

  • refer to EclipseSource instead of EclipseSource Munich
  • remove the link to the repository, enabling GH auto detection

Updates the LICENSE text to
- refer to EclipseSource instead of EclipseSource Munich
- remove the link to the repository, enabling GH auto detection
@sdirix
sdirix requested a review from lucas-koehler March 24, 2025 11:39
@netlify

netlify Bot commented Mar 24, 2025

Copy link
Copy Markdown

Deploy Preview for jsonforms-examples ready!

Name Link
🔨 Latest commit 7460677
🔍 Latest deploy log https://app.netlify.com/sites/jsonforms-examples/deploys/67e14457f3ce8a000836a807
😎 Deploy Preview https://deploy-preview-2431--jsonforms-examples.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@sdirix sdirix mentioned this pull request Mar 24, 2025
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 82.517%. remained the same
when pulling 7460677 on adapt-license-text
into 0628b99 on master.

@lucas-koehler lucas-koehler 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.

The change LGTM. Should we also adapt the license headers in the code files?

@sdirix

sdirix commented Mar 26, 2025

Copy link
Copy Markdown
Member Author

Good idea, let's handle that in a follow up at some point (low prio)

@sdirix
sdirix merged commit 9acd631 into master Mar 26, 2025
@sdirix
sdirix deleted the adapt-license-text branch March 26, 2025 10:46
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