Skip to content

Commit 2e2ff4c

Browse files
authored
Merge pull request #5 from chadhietala/ignore-mixins
Remove mixins doc for now
2 parents f6aa011 + 8330209 commit 2e2ff4c

File tree

2 files changed

+3
-72
lines changed

2 files changed

+3
-72
lines changed

README.md

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,9 @@ This adds more guard rails for Ember. Just because something "works" does not me
44

55
- Not adhereing to Data Down Actions Up
66
- Side-efffect programming
7-
- Usage of Mixins
8-
- Usage of Sub-Classes
9-
- Eager/Lazy injections
10-
- Warnings and deprecations are failing tests
7+
- Usage of observers
8+
- Usage of attrs.*
9+
- Computed properties without dependent keys
1110

1211
# Why
1312
As we've scaled out to over hundreds developers and hundrends of lines of code it has become clear that more "development time" guidance is required. While the guides do a good job about getting started, it's good to have some friction to make sure the app code is aligned with how Ember works.

guides/rules/mixins.md

Lines changed: 0 additions & 68 deletions
This file was deleted.

0 commit comments

Comments
 (0)