Skip to content

chore: add Apache-2.0 LICENSE (COR-13588) - #4

Open
Bradenream wants to merge 2 commits into
masterfrom
braden/add-apache-2-license/COR-0
Open

chore: add Apache-2.0 LICENSE (COR-13588)#4
Bradenream wants to merge 2 commits into
masterfrom
braden/add-apache-2-license/COR-0

Conversation

@Bradenream

@Bradenream Bradenream commented Aug 11, 2026

Copy link
Copy Markdown

Part of COR-13588 — shipping the vf CLI as an agent-era distribution channel.

Problem

The repository is public with no LICENSE file. gh api repos/voiceflow/cli --jq .license returns null, which leaves every install legally ambiguous — no grant of use, modification, or redistribution.

Change

  • Adds the verbatim Apache-2.0 text from apache.org/licenses/LICENSE-2.0.txt, with the appendix placeholders filled in as Copyright 2026 Voiceflow, Inc. Keeping the text verbatim is what lets GitHub's license detection recognise it.
  • Corrects the README badge, which advertised MIT. That badge sits above the first <!-- Start ... --> marker, outside Speakeasy-managed content, so the edit survives regeneration. Its link now points at the canonical opensource.org/license/Apache-2.0 rather than the redirecting /licenses/ form.

.goreleaser.yaml already globs LICENSE* into every release archive, so the file also starts shipping with the binaries as originally intended.

Apache-2.0 is a deliberate choice for this CLI — it adds an explicit patent grant over the MIT the badge previously claimed.

The repository is public with no LICENSE file, so `gh api repos/voiceflow/cli
--jq .license` returns null and every install is legally ambiguous. Adds the
verbatim Apache-2.0 text from apache.org, with the appendix placeholders filled
in as "Copyright 2026 Voiceflow, Inc." -- the entity string already used by
voiceflow/webchat.

.goreleaser.yaml already globs `LICENSE*` into every release archive, so the
file also starts shipping with the binaries as originally intended.

Also corrects the README badge, which advertised MIT. That badge sits above the
first `<!-- Start ... -->` marker, outside Speakeasy-managed content, so the
edit survives regeneration.

Note for reviewers: voiceflow/webchat, a public repo in this org, carries a
proprietary license. Apache-2.0 here is a deliberate choice for the CLI rather
than an inherited default, and is worth confirming with whoever owns licensing
policy before merge.
Copilot AI lite review requested due to automatic review settings August 11, 2026 21:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds an explicit open-source license to the repository and aligns the README’s license badge with that choice to reduce legal ambiguity and improve discoverability (e.g., GitHub license detection).

Changes:

  • Add an Apache License 2.0 LICENSE file (including the filled-in appendix notice).
  • Update the README license badge from MIT to Apache-2.0 and adjust the target link accordingly.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Updates the license badge to reflect Apache-2.0 instead of MIT.
LICENSE Introduces the Apache License 2.0 text as the repository’s license file.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread LICENSE Outdated
Bradenream added a commit that referenced this pull request Aug 26, 2026
- Restore the '# vf' H1: retitling it broke the generated table of
  contents' first anchor (#vf), which the TOC block still points at.
  The descriptive positioning moves to the line below, where it reads
  better anyway.
- Match PR #4's license-badge href exactly (opensource.org/license/...,
  the canonical 200; /licenses/ 301-redirects) so the two PRs cannot
  disagree about the final text.
- Drop '(recommended)' from the npm install heading — it is not a
  superlative worth making while the package is not yet published; the
  section is honest about what it does either way.
@Bradenream Bradenream changed the title chore: add Apache-2.0 LICENSE chore: add Apache-2.0 LICENSE (COR-13588) Aug 26, 2026
@linear-code

linear-code Bot commented Aug 26, 2026

Copy link
Copy Markdown

COR-13588

Copilot review: license scanners and GitHub's own license detection
expect the file to open with the license title. The upstream Apache-2.0
text starts at 'Apache License', not a blank line.
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