Skip to content

Commit 854a3c5

Browse files
keithamuskoddsson
andauthored
add note about polyfills
Co-authored-by: Kristján Oddsson <[email protected]>
1 parent 8c10be1 commit 854a3c5

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

README.md

+12
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,18 @@ npm install
3434
npm test
3535
```
3636

37+
## Contributing
38+
39+
### Adding polyfills
40+
41+
Please do not add any polyfills for ECMA features that are Stage 3 or below. We _only_ wish to polyfill features from ECMAScript that are Stage 4 (about to be included in a new years specification) or already specified.
42+
43+
### Removing polyfills
44+
45+
When our supported browsers (see github/github for details) all support a feature without polyfills, we can remove the polyfill. This code is designed to be kept lightweight, we do not want to ship dozens of kb of polyfills.
46+
47+
As a polyfill is removed, it may be worth adding feature detection to the `baseSupport` const, to ensure that our baseline moves with our browser support matrix.
48+
3749
## License
3850

3951
Distributed under the MIT license. See LICENSE for details.

0 commit comments

Comments
 (0)