Prepare the repository for public visibility - #127
Merged
Merged
Conversation
Remove every statement that Imp is private or needs GitHub credentials from the install paths a stranger reads first, add a NOTICE for the DSPy-ported files, pin every third-party GitHub Action to a commit SHA, and give the repository the contributor surface a public repo needs.
This was referenced Sep 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A stranger who opens this repository now finds an install line they can run,
a stated license position, a pull request template that asks for checks they
can actually run, and a bug template. Nothing tells them the repository is
private or that they need credentials, because neither is true.
Each change:
branch: "main"named as the way to getImp.ACPandImp.MCP.connect/2until a tag containing them is cut; the "Release state" paragraph at the top
now says the same thing.
runtime: false, so a release using the adapters needsapplications: [ex_mcp: :load], linked to the existing wording indocs/PRODUCTION_OPERATIONS.mdrather than duplicated.examples/deployment/README.mddrop the same privaterepository and credential statements; the five Livebooks' standalone install
comments drop the word "private" too.
decisions.md: the 2026-08-23 Hex ruling's status column records that therepository is public as of 2026-09-17 and that the Hex question is open.
NOTICEnames DSPy, its MIT license and copyright holder, and the twofiles ported from it (
lib/imp/optimizer/utils.ex,lib/imp/adapter/chat.ex).README points at it;
mix.exspackage_files/0ships it..github/pull_request_template.mdnow asks an outside contributor formix checkand a plain statement of what changed and how it was verified,and says provider-backed checks are the maintainer's to run.
ci.yml,evidence.ymland theelixir-setupcomposite action is pinned to a full commit SHA with its version tag in a
trailing comment, at the versions Dependabot Bump actions/cache from 4 to 6 #89, Bump dorny/paths-filter from 3 to 4 #90 and Bump actions/setup-python from 5 to 7 #91 propose
(actions/cache v6.1.0, dorny/paths-filter v4.0.3, actions/setup-python
v7.0.0), plus actions/checkout v7.0.1, erlef/setup-beam v1.24.1 and
denoland/setup-deno v2.0.5. Those three Dependabot pull requests can be
closed. The ci.yml header comment says the pins move with their comments.
.github/CODEOWNERS(* @deepfates) and.github/ISSUE_TEMPLATE/bug.md. No feature template.AGENTS.mdkeeps its design orientation but stands alone: no reference to aworkshop file or to sibling repositories, and it points at
CONTRIBUTING.mdfor the gates and
decisions.mdfor the rulings. I kept it rather thanfolding it into CONTRIBUTING.md because CONTRIBUTING.md covers setup and
gates while AGENTS.md is design context, and merging them would bury both.
What I could not do: neither open Dependabot alert can be fixed. The nltk
alert (high,
benchmarks/requirements-ifbench-parity.txt) and the acceleratealert (medium,
priv/trl_worker/uv.lock) both reportfirst_patched_version: null— no fixed release exists, and both pins are atthe top of the vulnerable range. Separately, the nltk pin is a source-exact
IFBench scorer environment that CI builds to reproduce upstream scores, so
even a patched release would have to be taken deliberately as a parity change,
not as a dependency bump.
One test assertion changed:
test/package_contract_test.exshad a test named"README gives the usable private source install"; the name is now "the usable
source install". Its assertions were already true and are unchanged. NOTICE
was added to that file's repository and product file lists so packaging is
asserted.
Verification, from this branch:
Files owned by the two sibling pull requests (the evidence-bookkeeping
deletion and the benchmark surface) are untouched.