search

LearnersGuild / echo

learning management system
MIT License
3 stars 31 forks source link

Patrick serafin 1082 #1100

Closed lumodon closed 6 years ago

lumodon commented 6 years ago

Fixes #1082

Overview

Linting warnings in master are currently at 420 warnings, many of them caused by not having statically defined import path due to autoloader. This PR removes autoloader and replaces many exports with explicitly defined exports. While we are at it, we also clean up a few stray simple linting warnings.

Data Model / DB Schema Changes

N/A

Environment / Configuration Changes

Removal of keymirror and autoloader

Notes

  1. This PR does not fix ALL the linting errors but cleans up the majority that are caused by autoloader. Need to file a new issue with specific errors being created due to 'import/named', 'prefer-default', and 'no-dynamic-import' linting warnings.
  2. This PR also includes the 'fs' utility program I used for rapidly creating all the static import/export index.js files under 'metautil/autoloadscript.js' -- this utility may end up being a one-time-used only file and never needed again, not sure if it should be included here or left out.
lumodon commented 6 years ago

Used wrong branch at beginning of PR - just force pushed correct branch