Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable strict mode #9

Open
EbonJaeger opened this issue Jun 14, 2018 · 0 comments
Open

Enable strict mode #9

EbonJaeger opened this issue Jun 14, 2018 · 0 comments
Labels
enhancement New feature or request

Comments

@EbonJaeger
Copy link
Owner

EbonJaeger commented Jun 14, 2018 •
edited
Loading

This will enforce the code to be more syntactically correct, as well as being more secure.

This may be a big project since some of the things currently done in the codebase are not allowed in strict mode. As such, this could result in major architectural changes.

See https://www.w3schools.com/js/js_strict.asp
@EbonJaeger EbonJaeger added the enhancement New feature or request label Jun 14, 2018
EbonJaeger added a commit that referenced this issue Jul 8, 2018
@EbonJaeger
Initial work to enable strict mode
055929b 
There is a lot of work left to do, mainly fixing all of the scope issues.

Ref #9
EbonJaeger added a commit that referenced this issue Jul 8, 2018
@EbonJaeger
Modularize the libs.js and enable strict mode in it
9c902e5 
There are still some scope issues to work out with this, but it should be mostly there.

Ref #9

Signed-off-by: Evan Maddock <[email protected]>
EbonJaeger added a commit that referenced this issue Jul 10, 2018
@EbonJaeger
Remove duplicate libs
e76be96 
Ref #9

Signed-off-by: Evan Maddock <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@EbonJaeger