Skip to content

Remove Apache 2 license from SKILL.md#22322

Merged
krrishdholakia merged 1 commit intoBerriAI:mainfrom
rasmi:main
Feb 28, 2026
Merged

Remove Apache 2 license from SKILL.md#22322
krrishdholakia merged 1 commit intoBerriAI:mainfrom
rasmi:main

Conversation

@rasmi
Copy link
Contributor

@rasmi rasmi commented Feb 27, 2026

#18232 (specifically 85c3333) added an Apache 2.0 license file in the repo, which conflicts with the MIT license in the root of the repo. This gets rid of it (and rebuilds the zip file that contained it).

@vercel
Copy link

vercel bot commented Feb 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview, Comment Feb 27, 2026 11:03pm

Request Review

@CLAassistant
Copy link

CLAassistant commented Feb 27, 2026

CLA assistant check
All committers have signed the CLA.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 27, 2026

Greptile Summary

Removes an Apache 2.0 license file (LICENSE.txt) and its reference in SKILL.md from the slack-gif-creator test skill data, resolving a license conflict with the repo's root MIT license introduced in #18232. The pre-built .zip file is also rebuilt to reflect these deletions.

  • Deleted tests/llm_translation/test_skills_data/slack-gif-creator/LICENSE.txt (Apache 2.0 license)
  • Removed license: Complete terms in LICENSE.txt frontmatter line from SKILL.md
  • Rebuilt slack-gif-creator.zip without the license file (16.7KB → 12.6KB)
  • No functional code changes; only test fixture data is affected
  • The e2e test (test_skills_e2e.py) dynamically creates its zip from the folder contents, so it will correctly reflect these changes

Confidence Score: 5/5

  • This PR is safe to merge — it only removes a conflicting license file from test fixture data with no functional code changes.
  • Score of 5 reflects that this is a minimal, purely housekeeping change: removing a license file and its reference from test data. No production code, logic, or behavior is affected. The change is clearly correct — removing an Apache 2.0 license that conflicts with the repo's MIT license.
  • No files require special attention.

Important Files Changed

Filename Overview
tests/llm_translation/test_skills_data/slack-gif-creator/LICENSE.txt Deleted Apache 2.0 license file that conflicted with the repo's MIT license. Correct removal.
tests/llm_translation/test_skills_data/slack-gif-creator/SKILL.md Removed the license: Complete terms in LICENSE.txt line from SKILL.md frontmatter, consistent with the LICENSE.txt deletion.
tests/llm_translation/test_skills_data/slack-gif-creator.zip Rebuilt zip file to exclude the removed LICENSE.txt. File size decreased from 16748 to 12585 bytes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["PR #18232 introduced<br/>Apache 2.0 LICENSE.txt"] --> B["Conflicts with repo's<br/>MIT License"]
    B --> C["This PR removes:<br/>LICENSE.txt + reference in SKILL.md"]
    C --> D["Rebuild slack-gif-creator.zip<br/>without LICENSE.txt"]
    D --> E["License conflict resolved<br/>Only MIT license remains"]
Loading

Last reviewed commit: 50c74a8

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@krrishdholakia krrishdholakia merged commit bffce84 into BerriAI:main Feb 28, 2026
25 of 29 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.

3 participants