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.
The text was updated successfully, but these errors were encountered:
Thanks for starting this. Some possible improvements:
test/
has copies of top-level files. Remove the cruft.logger
inlib/http.js
or via alib/logger.js
.bedrock-
prefix for the config namepace likebedrock-MODULE-http
? (Probably shouldn't sayprofile
as it does now.)bedrock-MODULE-http
orbedrock-FIXME-http
so it's a bit easier to see what to update.bedrock-module-template-http
? Or would a common-template
suffix work better?bedrock-module-http-template
,bedrock-module-template
,bedrock-web-module-template
etc.The text was updated successfully, but these errors were encountered: