You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
test/ has copies of top-level files. Remove the cruft.
Add a logger in lib/http.js or via a lib/logger.js.
Should we use a bedrock- prefix for the config namepace like bedrock-MODULE-http? (Probably shouldn't say profile as it does now.)
Perhaps in all the files, change module name to bedrock-MODULE-http or bedrock-FIXME-http so it's a bit easier to see what to update.
Should the template repo naming pattern be bedrock-module-template-http? Or would a common -template suffix work better? bedrock-module-http-template, bedrock-module-template, bedrock-web-module-template etc.
Thanks for starting this. Some possible improvements:
test/has copies of top-level files. Remove the cruft.loggerinlib/http.jsor via alib/logger.js.bedrock-prefix for the config namepace likebedrock-MODULE-http? (Probably shouldn't sayprofileas it does now.)bedrock-MODULE-httporbedrock-FIXME-httpso it's a bit easier to see what to update.bedrock-module-template-http? Or would a common-templatesuffix work better?bedrock-module-http-template,bedrock-module-template,bedrock-web-module-templateetc.