Skip to content

Cleanup/remove editors and tools build - #2114

Merged
svor merged 4 commits into
mainfrom
cleanup/remove-editors-and-tools-build
Aug 14, 2026
Merged

Cleanup/remove editors and tools build#2114
svor merged 4 commits into
mainfrom
cleanup/remove-editors-and-tools-build

Conversation

@svor

@svor svor commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

The plugin registry's sole purpose is now serving OpenVSX extensions for air-gapped/offline environments. This PR removes all editor definition infrastructure and Node.js build tooling that are no longer needed.

What's removed

  • Editor definitions: che-editors.yaml and v3/ directory (editor icons, plugins .htaccess)

  • Node.js tooling: tools/build/ npm package, root package.json, yarn.lock, .yarnrc.yml, .yarn/, .vscode/

  • Content publishing infrastructure: publish-pr-check-content, publish-content-push-main workflows, build/dev/publish-plugin-registry-to-gh-pages.sh, surge.sh deployment

  • Supporting workflows: typescript-pr-check, regen-yarn-lock-and-copy

  • Internal Red Hat downstream build references from README

  • release.yml: removed Python/yq setup, gh-pages publishing, publishToGhPages input

  • image-build-pr-check.yml: removed node/yarn setup, content artifact extraction, pull-request-info job

  • next-build.yml: removed node/yarn setup

  • README.md, CONTRIBUTE.md, .htaccess, dependabot.yml, .linelint.yml: cleaned up stale references

What's kept

  • OpenVSX server, PostgreSQL, httpd, import-vsix.sh, openvsx-sync.json — the core extension
    registry
  • .ci/openshift-ci/ test scaffolding (references editor devfiles but requires a broader CI
    rework)

svor and others added 2 commits August 14, 2026 16:41
…hing

The plugin registry now serves only OpenVSX extensions. Editor definitions
(che-editors.yaml) were the only consumer of the tools/build npm package
and the content publishing infrastructure (surge, gh-pages).

Removed:
- che-editors.yaml and v3/ directory (editor definitions and icons)
- tools/build/ npm package and all root yarn/npm files
- Workflows: typescript-pr-check, regen-yarn-lock-and-copy,
  publish-pr-check-content, publish-content-push-main
- build/dev/publish-plugin-registry-to-gh-pages.sh
- Editor content extraction from image-build-pr-check and release workflows
- Python/yq setup from release workflow (only needed for che-editors.yaml)
- .vscode/ settings (referenced .yarn SDKs)

Updated:
- build.sh: removed yarn/tools steps and --offline/--skip-digest-generation flags
- Dockerfile: removed COPY of output/v3, v3/plugins, v3/images
- Deploy manifests: health check path changed from /v3/plugins/ to /
- make-release.sh: replaced updateEditors with simple VERSION bump
- README.md, CONTRIBUTE.md, .htaccess, dependabot.yml, .linelint.yml

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
@svor
svor requested review from ibuziuk and vinokurig as code owners August 14, 2026 13:43
@openshift-ci

openshift-ci Bot commented Aug 14, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: svor

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@github-actions

Copy link
Copy Markdown

Click here to review and test in web IDE: Contribute

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
@tolusha

tolusha commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

⚠️ Warning: IDE/tool configuration files detected

This PR contains changes to files in directories that are typically not intended to be committed:

  • .vscode/extensions.json
  • .vscode/launch.json
  • .vscode/settings.json

Please verify these changes are intentional.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
@openshift-ci

openshift-ci Bot commented Aug 14, 2026

Copy link
Copy Markdown

@svor: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/v19-plugins-test-pr-check 1fb514c link true /test v19-plugins-test-pr-check
ci/prow/v19-images 1fb514c link true /test v19-images

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@svor
svor merged commit 3a7ab85 into main Aug 14, 2026
11 of 14 checks passed
@svor
svor deleted the cleanup/remove-editors-and-tools-build branch August 14, 2026 14:37
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.

2 participants