search

groovecoder / discord

GitHub webhook that analyzes pull requests and adds comments about incompatible CSS
Mozilla Public License 2.0
29 stars 13 forks source link

Improve file organization #134

Closed openjck closed 9 years ago

openjck commented 9 years ago

To keep the codebase well-organized, some files are moved into different directories based on their purpose.

Files that handle application paths are moved into a new routes/ directory as recommended by express-generator. [1]

Files that are included elsewhere as "helpers" are moved into a new lib/ directory. NPM [2], Gulp [3], and Stylus [4] follow the same pattern.

[1] http://expressjs.com/starter/generator.html [2] https://github.com/npm/npm/tree/master/lib [3] https://github.com/gulpjs/gulp/tree/master/lib [4] https://github.com/stylus/stylus/tree/master/lib

openjck commented 9 years ago

An easier way to review this might be to browse around the files on this branch and see what you think. The diff mostly shows path updates, which don't really illustrate what changed.

openjck commented 9 years ago

Rebased! :smile:

darkwing commented 9 years ago

Everything works. R+