Skip to content
This repository was archived by the owner on Jan 6, 2026. It is now read-only.

Remove unused scripts and update documentation#1756

Merged
NullVoxPopuli merged 3 commits intomainfrom
cleanup-unused-scripts
Jul 1, 2025
Merged

Remove unused scripts and update documentation#1756
NullVoxPopuli merged 3 commits intomainfrom
cleanup-unused-scripts

Conversation

@wycats
Copy link
Copy Markdown
Contributor

@wycats wycats commented Jul 1, 2025

Summary

This PR removes non-working scripts and their references from the codebase, cleaning up package.json and updating documentation to reflect the actual working commands.

Changes

  • Removed 7 non-working scripts from package.json:

    • browserstack:connect
    • browserstack:disconnect
    • browserstack:results
    • build:control
    • build:flags
    • test:browserstack
    • test:smoke
  • Removed unused dependencies:

    • ember-cli-browserstack
    • ember-cli
  • Updated README.md to use correct test commands:

    • Changed ember test --serverpnpm start
    • Updated test instructions to reflect current tooling

Test plan

  • Verified all referenced scripts were removed or non-existent
  • Ran CI preparation commands (pnpm lint:fix, pnpm repo:update:conventions, pnpm repo:update:metadata)
  • Checked for any remaining references to removed scripts
  • Confirmed working commands still function properly

wycats added 3 commits June 30, 2025 18:23
- Remove non-working scripts from package.json (browserstack, ember-cli commands)
- Delete browserstack-related files that are no longer used
- Update README.md to use correct test commands
- Update build-constraints.md to reflect current build commands
- Remove commented-out browserstack jobs from CI workflow
- Keep smoke tests as they are actively used
- Remove 7 non-working scripts from package.json (browserstack:*, build:control, build:flags, test:browserstack, test:smoke)
- Remove ember-cli dependencies that were no longer used
- Update README.md to use correct test commands (pnpm start instead of ember test --server)
- Update repository metadata after script cleanup
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 1, 2025

This PRmain
Dev
588K └─┬ .
169K   ├── runtime
160K   ├── syntax
100K   ├── compiler
 58K   ├── opcode-compiler
 27K   ├── manager
 24K   ├── validator
 11K   ├── program
8.9K   ├── reference
7.2K   ├── destroyable
6.3K   ├── util
4.3K   ├── node
3.4K   ├── global-context
2.5K   ├── wire-format
1.0K   ├── vm
969B   ├── encoder
844B   ├── vm-babel-plugins
606B   └── owner
588K └─┬ .
169K   ├── runtime
160K   ├── syntax
100K   ├── compiler
 58K   ├── opcode-compiler
 27K   ├── manager
 24K   ├── validator
 11K   ├── program
8.9K   ├── reference
7.2K   ├── destroyable
6.3K   ├── util
4.3K   ├── node
3.4K   ├── global-context
2.5K   ├── wire-format
1.0K   ├── vm
969B   ├── encoder
844B   ├── vm-babel-plugins
606B   └── owner
Prod
231K └─┬ .
 70K   ├── syntax
 63K   ├── runtime
 48K   ├── compiler
 18K   ├── opcode-compiler
7.9K   ├── manager
5.1K   ├── validator
4.8K   ├── program
3.6K   ├── reference
2.4K   ├── util
2.1K   ├── node
1.6K   ├── wire-format
1.5K   ├── destroyable
737B   ├── vm
594B   ├── global-context
516B   ├── encoder
469B   ├── vm-babel-plugins
155B   └── owner
231K └─┬ .
 70K   ├── syntax
 63K   ├── runtime
 48K   ├── compiler
 18K   ├── opcode-compiler
7.9K   ├── manager
5.1K   ├── validator
4.8K   ├── program
3.6K   ├── reference
2.4K   ├── util
2.1K   ├── node
1.6K   ├── wire-format
1.5K   ├── destroyable
737B   ├── vm
594B   ├── global-context
516B   ├── encoder
469B   ├── vm-babel-plugins
155B   └── owner

@NullVoxPopuli NullVoxPopuli merged commit 9d5fbfc into main Jul 1, 2025
10 of 11 checks passed
@NullVoxPopuli NullVoxPopuli deleted the cleanup-unused-scripts branch July 1, 2025 02:09
@github-actions github-actions Bot mentioned this pull request Jul 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants