Skip to content

fix(library): correct unlimited-execution wording in Apache 2.0 article#5685

Merged
waleedlatif1 merged 1 commit into
stagingfrom
library-apache2-admission-wording
Jul 15, 2026
Merged

fix(library): correct unlimited-execution wording in Apache 2.0 article#5685
waleedlatif1 merged 1 commit into
stagingfrom
library-apache2-admission-wording

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Follow-up to feat(library): add Apache 2.0 vs fair-code licensing article #5683, which merged before the review fix landed
  • Addresses Greptile's P2 on the licensing article: the "nothing in the software stops you" claim contradicted the default per-pod in-flight execution admission gate (ADMISSION_GATE_MAX_INFLIGHT, default 500)
  • Reworded to distinguish tunable burst-protection env defaults from license gates

Type of Change

  • Bug fix (content correction)

Testing

MDX frontmatter re-validated against the content schema; body compiles.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 15, 2026 6:32am

Request Review

@cursor

cursor Bot commented Jul 15, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation-only MDX edit with no runtime or security impact.

Overview
Content correction in the Apache 2.0 vs fair-code library article so self-hosting claims match how Sim actually behaves.

The self-hosting section no longer says nothing in the software stops you from running unlimited executions. It now says nothing in the license stops you, and adds that burst-protection defaults (e.g. a per-pod cap on concurrent in-flight executions) are environment variables you tune, not license gates. Wording also shifts execution meter to usage meter for consistency with the license-vs-software framing.

Reviewed by Cursor Bugbot for commit ca134c5. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR corrects the self-hosting limits described in the Apache 2.0 licensing article. The main change is:

  • Distinguishes configurable burst-protection defaults from license-based usage gates.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed content.
  • The revised wording matches the configurable per-pod admission control while keeping license limits separate from operational safeguards.

Important Files Changed

Filename Overview
apps/sim/content/library/apache-2-0-vs-fair-code/index.mdx Rewords the self-hosting paragraph to accurately describe the configurable per-pod execution safeguard.

Reviews (1): Last reviewed commit: "fix(library): correct unlimited-executio..." | Re-trigger Greptile

@waleedlatif1 waleedlatif1 merged commit eac7bf0 into staging Jul 15, 2026
13 checks passed
@waleedlatif1 waleedlatif1 deleted the library-apache2-admission-wording branch July 15, 2026 06:34
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.

1 participant