Skip to content
Snippets Groups Projects
Commit 6064322c authored by Blank_Alt's avatar Blank_Alt Committed by Pregmodder
Browse files

Improve contrib

parent 294876bf
No related branches found
No related tags found
1 merge request!10711Improve contrib
......@@ -141,15 +141,11 @@ const foo = {
New code should generally get organized into the `App` namespace. See [fc-js-init.js](js/002-config/fc-js-init.js) for a rough outline.
## JavaScript Features
* Avoid using very new JavaScript features
* Generally, we're currently targeting ECMAScript 2018, though we use a few widely-implemented ECMAScript 2019
features, like `globalThis`.
* Conversely, do use modern features, it's not 2010 anymore and we don't try to support Internet Explorer or anything
stupid like that.
* use `let`/`const` rather than `var`
* prefer fat arrow functions to inline long-form functions
* etc.
* Generally, we're currently targeting ECMAScript 2021.
* It's not 2010 anymore and we don't try to support Internet Explorer or anything stupid like that.
* use `let`/`const` rather than `var`
* prefer fat arrow functions to inline long-form functions
* etc.
## Code quality
......@@ -214,3 +210,4 @@ errors, it's not totally clean as is and there are a few false positives.
* [Classes in Game State](https://gitgud.io/pregmodfan/fc-pregmod/-/issues/696)
* [Self executing functions](https://gitgud.io/pregmodfan/fc-pregmod/-/issues/2325)
* [Sort a map](https://gitgud.io/pregmodfan/fc-pregmod/-/issues/2642)
* [How to create a Merge Request (MR)](https://gitgud.io/pregmodfan/fc-pregmod/-/issues/3903)
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment