Add comments

Highlighting some things that need to be addressed based on the
generated app.

Author: stevepolitodesign

templates/Gemfile.erb

@@ -23,6 +23,8 @@ gem "sprockets", "< 4"
 gem "title"
 gem "tzinfo-data", platforms: [:mingw, :x64_mingw, :mswin, :jruby]
 <%# TODO: Remove this, and consider passing an option to the `rails new` command for esbuild  %>
+<%# TODO: Maybe this is how we conditionally handle dependencies that need a bundler %>
+<%# unless options[:javascript] %>
 <%# gem "webpacker" %>
 # Rails 7
 gem "sprockets-rails"

templates/_javascript.html.erb

@@ -1,3 +1,5 @@
+<%# TODO: Replace this with javascript_tag %>
+<%# TODO: We might want to do conditionally do this though, incase the app is using webpackeh %>
 <%= javascript_pack_tag :application %>
 
 <%= yield :javascript %>

templates/application.scss

@@ -1,5 +1,6 @@
 @charset "utf-8";
 
+// TODO: We need to explore using cssbundling-rails with postcss to load this
 @import "normalize.css/normalize";
 
 @import "bourbon";