Skip to content

Remove: SDK Reference artifacts and apps/web#1199

Merged
jakubno merged 2 commits into
mainfrom
remove/sdk-reference-artifacts
Mar 17, 2026
Merged

Remove: SDK Reference artifacts and apps/web#1199
jakubno merged 2 commits into
mainfrom
remove/sdk-reference-artifacts

Conversation

@ben-fornefeld
Copy link
Copy Markdown
Member

@ben-fornefeld ben-fornefeld commented Mar 13, 2026

Note

Medium Risk
Medium risk because it deletes a large subtree (apps/web) and removes SDK-reference generation/commit steps from the package publish workflow, which may affect downstream docs/release expectations.

Overview
Removes the docs web app and generated SDK reference content. The PR deletes apps/web configs/scripts (Next.js/MDX setup, Sentry config, prebuild/sitemap generation) and removes the committed sdk-reference MDX pages.

Simplifies repo automation and ownership. The package publish workflow no longer generates/clones/commits SDK reference docs, CODEOWNERS drops web/docs ownership entries, and the root ESLint config removes @stylistic/ts in favor of the built-in semi rule.

Written by Cursor Bugbot for commit 4158d77. This will update automatically on new commits. Configure here.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 13, 2026

⚠️ No Changeset found

Latest commit: 4158d77

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR
Some errors occurred when validating the changesets config:
The package or glob expression "e2b-docs" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 13, 2026

Package Artifacts

Built from 341dd79. Download artifacts from this workflow run.

JS SDK (e2b@2.14.2-remove-sdk-reference-artifacts.0):

npm install ./e2b-2.14.2-remove-sdk-reference-artifacts.0.tgz

CLI (@e2b/cli@2.8.2-remove-sdk-reference-artifacts.0):

npm install ./e2b-cli-2.8.2-remove-sdk-reference-artifacts.0.tgz

Python SDK (e2b==2.15.2+remove-sdk-reference-artifacts):

pip install ./e2b-2.15.2+remove.sdk.reference.artifacts-py3-none-any.whl

Copy link
Copy Markdown
Contributor

@matthewlouisbrockman matthewlouisbrockman left a comment

Choose a reason for hiding this comment

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

looks good - don't think we need an version bumps because it's purely rm the extra docs

@jakubno jakubno merged commit 089b8b9 into main Mar 17, 2026
27 of 35 checks passed
@jakubno jakubno deleted the remove/sdk-reference-artifacts branch March 17, 2026 17:08
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