Skip to content

Remove all view mixins#20926

Merged
wagenet merged 5 commits intoemberjs:mainfrom
wagenet:kill-view-mixins-main
Jun 11, 2025
Merged

Remove all view mixins#20926
wagenet merged 5 commits intoemberjs:mainfrom
wagenet:kill-view-mixins-main

Conversation

@wagenet
Copy link
Member

@wagenet wagenet commented Jun 10, 2025

These are all internal so this should be able to merge without any deprecation work.

@wagenet wagenet mentioned this pull request Jun 10, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jun 10, 2025

Development Assets

Diff

--- main/out.txt	2025-06-10 16:53:24.000000000 +0000
+++ pr/./pr-15573299928/out.txt	2025-06-11 00:40:58.000000000 +0000
@@ -1,98 +1,98 @@
- 2.2M └─┬ .
-1014K   ├─┬ @ember
- 205K   │ ├─┬ -internals
-  69K   │ │ ├─┬ views
-  64K   │ │ │ └─┬ lib
-  23K   │ │ │   ├── mixins
-  22K   │ │ │   ├── system
-  10K   │ │ │   ├── views
- 4.3K   │ │ │   └── compat
-  35K   │ │ ├─┬ runtime
-  30K   │ │ │ └─┬ lib
-  21K   │ │ │   ├── mixins
- 5.7K   │ │ │   └── ext
-  26K   │ │ ├─┬ meta
-  21K   │ │ │ └── lib
-  11K   │ │ ├── owner
- 9.4K   │ │ ├── deprecations
- 7.4K   │ │ ├── metal
- 7.0K   │ │ ├── string
- 5.0K   │ │ ├── glimmer
- 4.9K   │ │ ├── utils
- 4.9K   │ │ ├── routing
- 4.5K   │ │ ├── error-handling
- 4.5K   │ │ ├── utility-types
- 4.2K   │ │ ├── container
- 4.2K   │ │ ├── browser-environment
- 4.1K   │ │ └── environment
- 183K   │ ├─┬ routing
-  28K   │ │ └── lib
- 149K   │ ├─┬ object
-  66K   │ │ └─┬ lib
-  62K   │ │   └── computed
- 114K   │ ├─┬ template-compiler
- 109K   │ │ └─┬ lib
-  20K   │ │   ├── plugins
- 4.6K   │ │   ├── system
- 4.1K   │ │   └── -internal
-  66K   │ ├─┬ application
- 5.6K   │ │ └── lib
-  52K   │ ├─┬ debug
-  21K   │ │ └── lib
-  38K   │ ├─┬ array
- 4.9K   │ │ └── lib
-  31K   │ ├─┬ engine
- 4.7K   │ │ └── lib
-  27K   │ ├── runloop
-  22K   │ ├─┬ utils
-  18K   │ │ └── lib
-  20K   │ ├── helper
-  11K   │ ├── destroyable
- 9.8K   │ ├── instrumentation
- 9.4K   │ ├── controller
- 7.4K   │ ├── service
- 7.2K   │ ├── owner
- 6.2K   │ ├── component
- 5.6K   │ ├── canary-features
- 5.5K   │ ├── modifier
- 5.1K   │ ├── template-compilation
- 5.0K   │ ├── enumerable
- 5.0K   │ ├── test
- 4.4K   │ ├── template
- 4.4K   │ ├── renderer
- 4.2K   │ ├── deprecated-features
- 4.1K   │ ├── template-factory
- 4.1K   │ └── version
- 708K   ├── shared-chunks
- 384K   ├─┬ @glimmer
- 166K   │ ├── runtime
-  60K   │ ├── opcode-compiler
-  30K   │ ├── manager
-  22K   │ ├── validator
-  14K   │ ├── program
-  12K   │ ├── reference
-  11K   │ ├── destroyable
-  10K   │ ├─┬ tracking
- 4.4K   │ │ └── primitives
-  10K   │ ├── util
- 8.1K   │ ├── node
- 7.3K   │ ├── global-context
- 6.4K   │ ├── wire-format
- 5.0K   │ ├── vm
- 4.9K   │ ├── encoder
- 4.6K   │ ├── owner
- 4.1K   │ └── env
-  60K   ├─┬ ember-testing
-  56K   │ └─┬ lib
-  14K   │   ├── test
-  14K   │   ├── helpers
-  10K   │   ├── ext
- 6.4K   │   └── adapters
-  31K   ├── backburner.js
-  25K   ├── ember
-  24K   ├── route-recognizer
-  18K   ├─┬ @simple-dom
-  14K   │ └── document
- 9.2K   ├── dag-map
- 4.3K   ├── rsvp
- 4.3K   └── router_js
+2.2M └─┬ .
+996K   ├─┬ @ember
+187K   │ ├─┬ -internals
+ 50K   │ │ ├─┬ views
+ 46K   │ │ │ └─┬ lib
+ 21K   │ │ │   ├── system
+ 10K   │ │ │   ├── views
+5.6K   │ │ │   ├── mixins
+4.3K   │ │ │   └── compat
+ 35K   │ │ ├─┬ runtime
+ 30K   │ │ │ └─┬ lib
+ 21K   │ │ │   ├── mixins
+5.7K   │ │ │   └── ext
+ 26K   │ │ ├─┬ meta
+ 21K   │ │ │ └── lib
+ 11K   │ │ ├── owner
+9.4K   │ │ ├── deprecations
+7.4K   │ │ ├── metal
+7.0K   │ │ ├── string
+5.0K   │ │ ├── glimmer
+4.9K   │ │ ├── utils
+4.9K   │ │ ├── routing
+4.5K   │ │ ├── error-handling
+4.5K   │ │ ├── utility-types
+4.2K   │ │ ├── container
+4.2K   │ │ ├── browser-environment
+4.1K   │ │ └── environment
+183K   │ ├─┬ routing
+ 28K   │ │ └── lib
+149K   │ ├─┬ object
+ 66K   │ │ └─┬ lib
+ 62K   │ │   └── computed
+114K   │ ├─┬ template-compiler
+109K   │ │ └─┬ lib
+ 20K   │ │   ├── plugins
+4.6K   │ │   ├── system
+4.1K   │ │   └── -internal
+ 65K   │ ├─┬ application
+5.6K   │ │ └── lib
+ 52K   │ ├─┬ debug
+ 21K   │ │ └── lib
+ 38K   │ ├─┬ array
+4.9K   │ │ └── lib
+ 31K   │ ├─┬ engine
+4.7K   │ │ └── lib
+ 27K   │ ├── runloop
+ 22K   │ ├─┬ utils
+ 18K   │ │ └── lib
+ 20K   │ ├── helper
+ 11K   │ ├── destroyable
+9.8K   │ ├── instrumentation
+9.4K   │ ├── controller
+7.4K   │ ├── service
+7.2K   │ ├── owner
+6.2K   │ ├── component
+5.6K   │ ├── canary-features
+5.5K   │ ├── modifier
+5.1K   │ ├── template-compilation
+5.0K   │ ├── enumerable
+5.0K   │ ├── test
+4.4K   │ ├── template
+4.4K   │ ├── renderer
+4.2K   │ ├── deprecated-features
+4.1K   │ ├── template-factory
+4.1K   │ └── version
+722K   ├── shared-chunks
+384K   ├─┬ @glimmer
+166K   │ ├── runtime
+ 60K   │ ├── opcode-compiler
+ 30K   │ ├── manager
+ 22K   │ ├── validator
+ 14K   │ ├── program
+ 12K   │ ├── reference
+ 11K   │ ├── destroyable
+ 10K   │ ├─┬ tracking
+4.4K   │ │ └── primitives
+ 10K   │ ├── util
+8.1K   │ ├── node
+7.3K   │ ├── global-context
+6.4K   │ ├── wire-format
+5.0K   │ ├── vm
+4.9K   │ ├── encoder
+4.6K   │ ├── owner
+4.1K   │ └── env
+ 60K   ├─┬ ember-testing
+ 55K   │ └─┬ lib
+ 14K   │   ├── test
+ 14K   │   ├── helpers
+ 10K   │   ├── ext
+6.4K   │   └── adapters
+ 31K   ├── backburner.js
+ 25K   ├── ember
+ 24K   ├── route-recognizer
+ 18K   ├─┬ @simple-dom
+ 14K   │ └── document
+9.2K   ├── dag-map
+4.3K   ├── rsvp
+4.3K   └── router_js

Details

This PRmain
Dev
2.2M └─┬ .
996K   ├─┬ @ember
187K   │ ├─┬ -internals
 50K   │ │ ├─┬ views
 46K   │ │ │ └─┬ lib
 21K   │ │ │   ├── system
 10K   │ │ │   ├── views
5.6K   │ │ │   ├── mixins
4.3K   │ │ │   └── compat
 35K   │ │ ├─┬ runtime
 30K   │ │ │ └─┬ lib
 21K   │ │ │   ├── mixins
5.7K   │ │ │   └── ext
 26K   │ │ ├─┬ meta
 21K   │ │ │ └── lib
 11K   │ │ ├── owner
9.4K   │ │ ├── deprecations
7.4K   │ │ ├── metal
7.0K   │ │ ├── string
5.0K   │ │ ├── glimmer
4.9K   │ │ ├── utils
4.9K   │ │ ├── routing
4.5K   │ │ ├── error-handling
4.5K   │ │ ├── utility-types
4.2K   │ │ ├── container
4.2K   │ │ ├── browser-environment
4.1K   │ │ └── environment
183K   │ ├─┬ routing
 28K   │ │ └── lib
149K   │ ├─┬ object
 66K   │ │ └─┬ lib
 62K   │ │   └── computed
114K   │ ├─┬ template-compiler
109K   │ │ └─┬ lib
 20K   │ │   ├── plugins
4.6K   │ │   ├── system
4.1K   │ │   └── -internal
 65K   │ ├─┬ application
5.6K   │ │ └── lib
 52K   │ ├─┬ debug
 21K   │ │ └── lib
 38K   │ ├─┬ array
4.9K   │ │ └── lib
 31K   │ ├─┬ engine
4.7K   │ │ └── lib
 27K   │ ├── runloop
 22K   │ ├─┬ utils
 18K   │ │ └── lib
 20K   │ ├── helper
 11K   │ ├── destroyable
9.8K   │ ├── instrumentation
9.4K   │ ├── controller
7.4K   │ ├── service
7.2K   │ ├── owner
6.2K   │ ├── component
5.6K   │ ├── canary-features
5.5K   │ ├── modifier
5.1K   │ ├── template-compilation
5.0K   │ ├── enumerable
5.0K   │ ├── test
4.4K   │ ├── template
4.4K   │ ├── renderer
4.2K   │ ├── deprecated-features
4.1K   │ ├── template-factory
4.1K   │ └── version
722K   ├── shared-chunks
384K   ├─┬ @glimmer
166K   │ ├── runtime
 60K   │ ├── opcode-compiler
 30K   │ ├── manager
 22K   │ ├── validator
 14K   │ ├── program
 12K   │ ├── reference
 11K   │ ├── destroyable
 10K   │ ├─┬ tracking
4.4K   │ │ └── primitives
 10K   │ ├── util
8.1K   │ ├── node
7.3K   │ ├── global-context
6.4K   │ ├── wire-format
5.0K   │ ├── vm
4.9K   │ ├── encoder
4.6K   │ ├── owner
4.1K   │ └── env
 60K   ├─┬ ember-testing
 55K   │ └─┬ lib
 14K   │   ├── test
 14K   │   ├── helpers
 10K   │   ├── ext
6.4K   │   └── adapters
 31K   ├── backburner.js
 25K   ├── ember
 24K   ├── route-recognizer
 18K   ├─┬ @simple-dom
 14K   │ └── document
9.2K   ├── dag-map
4.3K   ├── rsvp
4.3K   └── router_js
 2.2M └─┬ .
1014K   ├─┬ @ember
 205K   │ ├─┬ -internals
  69K   │ │ ├─┬ views
  64K   │ │ │ └─┬ lib
  23K   │ │ │   ├── mixins
  22K   │ │ │   ├── system
  10K   │ │ │   ├── views
 4.3K   │ │ │   └── compat
  35K   │ │ ├─┬ runtime
  30K   │ │ │ └─┬ lib
  21K   │ │ │   ├── mixins
 5.7K   │ │ │   └── ext
  26K   │ │ ├─┬ meta
  21K   │ │ │ └── lib
  11K   │ │ ├── owner
 9.4K   │ │ ├── deprecations
 7.4K   │ │ ├── metal
 7.0K   │ │ ├── string
 5.0K   │ │ ├── glimmer
 4.9K   │ │ ├── utils
 4.9K   │ │ ├── routing
 4.5K   │ │ ├── error-handling
 4.5K   │ │ ├── utility-types
 4.2K   │ │ ├── container
 4.2K   │ │ ├── browser-environment
 4.1K   │ │ └── environment
 183K   │ ├─┬ routing
  28K   │ │ └── lib
 149K   │ ├─┬ object
  66K   │ │ └─┬ lib
  62K   │ │   └── computed
 114K   │ ├─┬ template-compiler
 109K   │ │ └─┬ lib
  20K   │ │   ├── plugins
 4.6K   │ │   ├── system
 4.1K   │ │   └── -internal
  66K   │ ├─┬ application
 5.6K   │ │ └── lib
  52K   │ ├─┬ debug
  21K   │ │ └── lib
  38K   │ ├─┬ array
 4.9K   │ │ └── lib
  31K   │ ├─┬ engine
 4.7K   │ │ └── lib
  27K   │ ├── runloop
  22K   │ ├─┬ utils
  18K   │ │ └── lib
  20K   │ ├── helper
  11K   │ ├── destroyable
 9.8K   │ ├── instrumentation
 9.4K   │ ├── controller
 7.4K   │ ├── service
 7.2K   │ ├── owner
 6.2K   │ ├── component
 5.6K   │ ├── canary-features
 5.5K   │ ├── modifier
 5.1K   │ ├── template-compilation
 5.0K   │ ├── enumerable
 5.0K   │ ├── test
 4.4K   │ ├── template
 4.4K   │ ├── renderer
 4.2K   │ ├── deprecated-features
 4.1K   │ ├── template-factory
 4.1K   │ └── version
 708K   ├── shared-chunks
 384K   ├─┬ @glimmer
 166K   │ ├── runtime
  60K   │ ├── opcode-compiler
  30K   │ ├── manager
  22K   │ ├── validator
  14K   │ ├── program
  12K   │ ├── reference
  11K   │ ├── destroyable
  10K   │ ├─┬ tracking
 4.4K   │ │ └── primitives
  10K   │ ├── util
 8.1K   │ ├── node
 7.3K   │ ├── global-context
 6.4K   │ ├── wire-format
 5.0K   │ ├── vm
 4.9K   │ ├── encoder
 4.6K   │ ├── owner
 4.1K   │ └── env
  60K   ├─┬ ember-testing
  56K   │ └─┬ lib
  14K   │   ├── test
  14K   │   ├── helpers
  10K   │   ├── ext
 6.4K   │   └── adapters
  31K   ├── backburner.js
  25K   ├── ember
  24K   ├── route-recognizer
  18K   ├─┬ @simple-dom
  14K   │ └── document
 9.2K   ├── dag-map
 4.3K   ├── rsvp
 4.3K   └── router_js

Copy link
Contributor

@NullVoxPopuli NullVoxPopuli left a comment

Choose a reason for hiding this comment

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

Seems fine -- tho I see a lot more stuff to delete!

@wagenet wagenet merged commit ad60313 into emberjs:main Jun 11, 2025
28 checks passed
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