search

bower / registry

The Bower registry
https://registry.bower.io/packages
MIT License
292 stars 66 forks source link

Small refactor for the routes file #127

Closed noamokman closed 8 years ago

noamokman commented 9 years ago

Hi there, Just wanted to suggest this way of handling the excess to the express app from the routes file. I made the routes file expose a function that accepts the express app. Therefore we can avoid this line:

var app = module.parent.exports.app;
samccone commented 9 years ago

:clap: so much better thanks @noamokman

noamokman commented 9 years ago

Happy to help :+1:

noamokman commented 9 years ago

any reason this isn't merged in yet?

noamokman commented 8 years ago

updated branch to prevent merge confilcts

sheerun commented 8 years ago

Rebased and merged to master. Thank you