suspenders 1.50.0 → 1.54.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8da6287b8db00f55d3055a907f8c8e38b449ef15d3f98c1e898770594a8b9134
-  data.tar.gz: 54e3a53194b0b996d5971681d44269122bbf181c8d34f1638e2255592dcbf497
+  metadata.gz: a1c3a5fd298d2c49ceed3d4b3c79d99590aae75259c0f4ec6f06bd6f244336d3
+  data.tar.gz: c880d208a127a68da87b63a33da39fade1abf8e6c6d6b754806771be68e23e03
 SHA512:
-  metadata.gz: 407d7b2087c72518a79276894c1b7c4c7d8ebd9d0d6d5cbbb2f5b068935d1543a976f86bba1314bcf698e4cf1cb54ac279a6c21f0dd5ee485c0f8357f82133f5
-  data.tar.gz: b1bdb1c214097d96bb793614c822ddc20e090fe4cffc11b71841534c20d0108d03bb9fafaebc905b5b5923a26a5e364a2b6de21c03d41f9e2e523f4be6a4eeab
+  metadata.gz: 35d40b50ca80e2248efe6cc0c5ac5b87eff319aa6638e9293bafbb798081a1143b7a2d0e5907261bf68ceece79f482471c0762782a6d4cfebc69b0871fe8ea28
+  data.tar.gz: d02195af5d9673603233ce092cae10b4a2e2cf49697d8129c808a378d743d008f9046787b53ec050ec9a4450433fb063a7fbc4308fb5c16efc17d3ed73de291a

data/.ruby-version

@@ -1 +1 @@
-2.5.3
+2.6.6

data/.travis.yml

@@ -1,13 +1,15 @@
 language: ruby
-rvm: 2.5.3
+rvm: 2.6.6
 cache: bundler
-sudo: false
 before_install:
-  - "echo '--colour' > ~/.rspec"
+  - nvm install 13.12.0
   - git config --global user.name 'Travis CI'
   - git config --global user.email 'travis-ci@example.com'
-  - gem install bundler
   - gem update --system
+  - gem install bundler
+  - bundler config --global disable_platform_warnings true
 install: bundle install
 notifications:
   email: false
+services:
+  - postgresql

data/CONTRIBUTING.md

@@ -1,7 +1,7 @@
 # Contributing
 
-We love pull requests from everyone. By participating in this project, you agree
-to abide by the thoughtbot [code of conduct].
+We love pull requests from everyone. By participating in this project, you
+agree to abide by the thoughtbot [code of conduct].
 
 [code of conduct]: https://thoughtbot.com/open-source-code-of-conduct
 
@@ -17,17 +17,15 @@ Make sure the tests pass:
 
     rake
 
-Make your change.
-Write tests.
-Follow our [style guide][style].
-Make the tests pass:
+Make your change. Write tests. Follow our [style guide][style]. Make the tests
+pass:
 
 [style]: https://github.com/thoughtbot/guides/tree/master/style
 
     rake
 
-Mention how your changes affect the project to other developers and users in the
-`NEWS.md` file.
+Mention how your changes affect the project to other developers and users in
+the `NEWS.md` file.
 
 Write a [good commit message][commit].
 Push to your fork.
@@ -36,19 +34,21 @@ Push to your fork.
 [commit]: http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
 [pr]: https://github.com/thoughtbot/suspenders/compare/
 
-If [Hound] catches style violations,
-fix them.
+If [Hound] catches style violations, fix them.
 
 [hound]: https://houndci.com
 
-Wait for us.
-We try to at least comment on pull requests within one business day.
-We may suggest changes.
+Wait for us. We try to at least comment on pull requests within one business
+day. We may suggest changes.
+
+## Generators
+
+All new contributions must be within the generator framework, as described in
+[our goals](GOALS.md).
 
 ## Versions
 
-To update the Ruby version,
-change `.ruby-version` and `.travis.yml`.
+To update the Ruby version, change `.ruby-version` and `.travis.yml`.
 
 If you see this error while running tests:
 

data/GOALS.md

@@ -0,0 +1,65 @@
+# Goals
+
+Suspenders is an opinionated platform for adding functionality to a Rails app.
+It encodes thoughtbot's opinions and decisions in tooling and best practices.
+
+We are transitioning to a library of generators. All new functionality must be
+in the form of a well-tested, self-contained generator that fits within the
+Suspenders framework.
+
+## History and transitions
+
+In the beginning suspenders was a git repo that we would clone then use as a
+template for starting a new Rails app.
+
+It then became a command-line tool that would generate a Rails app then modify
+the files in place. Over time, and with intention, most of the modifications
+were focused on modifying the Gemfile.
+
+This was identified as unscalable and we have been slowing been moving in a new
+direction: a library of generators.
+
+## The future
+
+The problem with a script that adds a bunch of gem requirements to the Gemfile
+is: what if you don't want all those gems? Suspenders adds gem requirements and
+then configures each one. On a typical project we would run Suspenders then
+spend the rest of the day un-doing the parts we don't want.
+
+In 2015, [PR #511], we discussed what we want: a gem that is a library of
+generators that we can run on demand. The ideal future workflow goes like this:
+
+[PR #511]: https://github.com/thoughtbot/suspenders/pull/511
+
+1. `rails new my-cool-project && cd my-cool-project`
+2. `echo "gem 'suspenders', group: :development" >> Gemfile && bundle`
+3. `rails g suspenders:ci`
+4. Repeat with other generators as needed.
+
+In this future you would use `rails new` as normal and instead add Suspenders
+as a development gem. This gives you access to opinionated generators for
+adding (and removing!) whole functionality, but on demand and with your
+knowledge.
+
+The generators are all named to generically describe what functionality they
+add. In this way we can change our opinions on the implementation without
+needing to know new generator names. For example, `suspenders:ci` could
+configure the app for CircleCI now, but down the road we may decide to switch
+everything to Jenkins; we could update the implementation of the generator
+without changing its name.
+
+## From here to there
+
+The transition looks like this:
+
+1. Introduce a `suspenders:all` generator that runs Suspenders against an
+   existing Rails app.
+2. Change the Suspenders program to create a Rails app, add Suspenders to the
+   Gemfile, and run `suspenders:all` inside of it.
+3. Slowly split functionality out from the "all" generator and into smaller,
+   well-tested, self-contained generators. The "all" generator invokes these
+   generators.
+4. Once that is completed -- once "all" _only_ calls other generators -- then
+   remove the Suspenders program and only distribute the gem.
+
+We are on step (3), and it's a big one. Come help out!

data/NEWS.md

@@ -1,3 +1,58 @@
+Unreleased
+
+1.54.1 (June 30, 2020)
+
+* Fixed: invalid Gemfile entry for bundler-audit
+* Fixed: Deprecation warning for `Bundler.with_clean_env`
+
+1.54.0 (June 24, 2020)
+
+* New generator: `suspenders:single_redirect` for setting up `Rack::CanonicalHost`
+* New generator: `suspenders:production:compression` for setting up `Rack::Deflater`
+* New generator: `suspenders:preloader` for managing spring
+* New generator: `suspenders:advisories` for installing bundler-audit
+* New generator: `suspenders:profiler` for setting up rack\_mini\_profiler
+* New generator: `suspenders:runner` for making an app runnable locally
+* Added: generator descriptions
+* Added: Heroku release phase for running database migrations
+* Added: automatic buildpack configuration for Heroku
+* Added: system test configuration to opt into JavaScript as needed
+* Added: spring-watcher-listen gem
+* Changed: from `chromedriver-helper` to `webdrivers`
+* Changed: replace `heroku join` calls with `heroku apps:info` in `bin/setup`
+* Changed: check environment instead of `DATABASE\_URL` in test helper
+* Fixed: `ExpandJson` merging
+* Fixed: spacing in the generated config file
+* Upgraded: bitters to version 2.x
+* Upgraded: bourbon to version 6.x
+* Upgraded: Ruby to version 2.6.6
+* Removed: neat gem
+* Removed: ctags configuration
+* Removed: custom placeholder directories
+* Removed: customization for error pages
+* Removed: custom Puma config
+
+1.53.0 (August 23, 2019)
+* Upgraded: Rails 6.0.
+* New generator: `suspenders:inline_svg` for setting up the inline_svg gem.
+* Changed: silence Puma's startup messages in JS specs.
+* Changed: improve chromedriver configuration.
+* Added: documentation for deploying to Heroku.
+* Fixed: indentation in production config file.
+
+1.52.0 (June 7, 2019)
+* Changed: Setup system tests instead of feature specs
+* Upgraded: Rails 5.2.3
+
+1.51.0 (April 26, 2019)
+
+* Changed: from sass-rails to sassc-rails.
+* Upgraded: Ruby 2.6.3.
+* Fixed: Avoid installing autoprefixer-rails in api mode
+* New generator: `suspenders:stylelint` for setting up stylelint.
+* New generator: `suspenders:production:manifest` for app.json.
+* New generator: `suspenders:production:deployment` for bin/deploy script.
+
 1.50.0 (December 28, 2018)
 
 * Removed: jquery-rails.

data/README.md

@@ -37,8 +37,7 @@ It includes application gems like:
 * [Delayed Job](https://github.com/collectiveidea/delayed_job) for background
   processing
 * [High Voltage](https://github.com/thoughtbot/high_voltage) for static pages
-* [Honeybadger](https://honeybadger.io) for exception notification
-* [Neat](https://github.com/thoughtbot/neat) for semantic grids
+* [Honeybadger](https://www.honeybadger.io/?affiliate=A43uwl) for exception notification
 * [Normalize](https://necolas.github.io/normalize.css/) for resetting browser styles
 * [Oj](http://www.ohler.com/oj/)
 * [Postgres](https://github.com/ged/ruby-pg) for access to the Postgres database
@@ -53,7 +52,6 @@ It includes application gems like:
 * [Skylight](https://www.skylight.io/) for monitoring performance
 * [Title](https://github.com/calebthompson/title) for storing titles in
   translations
-* [Puma](https://github.com/puma/puma) to serve HTTP requests
 
 And development gems like:
 
@@ -101,6 +99,7 @@ Suspenders also comes with:
   environments
 * Configuration for [CircleCI][circle] Continuous Integration (tests)
 * Configuration for [Hound][hound] Continuous Integration (style)
+* Configuration for [stylelint][stylelint]
 * The analytics adapter [Segment][segment] (and therefore config for Google
   Analytics, Intercom, Facebook Ads, Twitter Ads, etc.)
 
@@ -111,10 +110,13 @@ Suspenders also comes with:
 [i18n]: https://github.com/thoughtbot/suspenders/pull/304
 [circle]: https://circleci.com/docs
 [hound]: https://houndci.com
+[stylelint]: https://stylelint.io/
 [segment]: https://segment.com
 
 ## Heroku
 
+Read the documentation on [deploying to Heroku][heroku deploy]
+
 You can optionally create Heroku staging and production apps:
 
     suspenders app --heroku true
@@ -129,6 +131,7 @@ This:
 * Schedules automated backups for 10AM UTC for both `staging` and `production`
 
 [Heroku Pipeline]: https://devcenter.heroku.com/articles/pipelines
+[heroku deploy]: https://github.com/thoughtbot/suspenders/blob/master/docs/heroku_deploy.md
 
 You can optionally specify alternate Heroku flags:
 
@@ -220,6 +223,6 @@ See [our other projects][community].
 We are [available for hire][hire].
 
 [thoughtbot]: https://thoughtbot.com?utm_source=github
-[thoughtbot-logo]: http://presskit.thoughtbot.com/images/thoughtbot-logo-for-readmes.svg
+[thoughtbot-logo]: https://thoughtbot.com/brand_assets/93:44.svg
 [community]: https://thoughtbot.com/community?utm_source=github
 [hire]: https://thoughtbot.com?utm_source=github

data/docs/heroku_deploy.md

@@ -0,0 +1,19 @@
+# Deploying to Heroku
+
+## Setup
+
+Deploying to Heroku requires two additional steps:
+
+1. Set the following environment variables:
+
+- `ASSET_HOST`: `siteURL.herokuapp.com`
+- `APPLICATION_HOST`: `siteURL.herokuapp.com`
+- `SMTP_ADDRESS`: `smtp.example.com`
+- `SMTP_DOMAIN`: `example.com`
+- `SMTP_USERNAME`: `username`
+- `SMTP_PASSWORD`: `password`
+- `AUTO_MIGRATE_DB`: `true`
+
+## Execution
+
+- Use the `./bin/deploy` convention for deploying to Heroku

data/lib/suspenders.rb

@@ -1,7 +1,9 @@
 require "suspenders/version"
+require "suspenders/generators/advisories_generator"
 require "suspenders/generators/app_generator"
 require "suspenders/generators/static_generator"
 require "suspenders/generators/stylesheet_base_generator"
+require "suspenders/generators/stylelint_generator"
 require "suspenders/generators/forms_generator"
 require "suspenders/generators/ci_generator"
 require "suspenders/generators/db_optimizations_generator"
@@ -13,9 +15,17 @@ require "suspenders/generators/views_generator"
 require "suspenders/generators/js_driver_generator"
 require "suspenders/generators/json_generator"
 require "suspenders/generators/testing_generator"
+require "suspenders/generators/inline_svg_generator"
+require "suspenders/generators/preloader_generator"
+require "suspenders/generators/profiler_generator"
+require "suspenders/generators/runner_generator"
 require "suspenders/generators/production/force_tls_generator"
+require "suspenders/generators/production/compression_generator"
 require "suspenders/generators/production/email_generator"
 require "suspenders/generators/production/timeout_generator"
+require "suspenders/generators/production/deployment_generator"
+require "suspenders/generators/production/manifest_generator"
+require "suspenders/generators/production/single_redirect"
 require "suspenders/generators/staging/pull_requests_generator"
 require "suspenders/actions"
 require "suspenders/adapters/heroku"

data/lib/suspenders/actions.rb

@@ -39,7 +39,7 @@ module Suspenders
       end
 
       def invoke!
-        write_out { |existing_json| existing_json.merge(data) }
+        write_out { |existing_json| existing_json.deep_merge(data) }
       end
 
       def revoke!
@@ -60,7 +60,7 @@ module Suspenders
       end
 
       def existing_json
-        JSON.parse(IO.read(destination_file))
+        JSON.parse(IO.read(destination_file), symbolize_names: true)
       rescue Errno::ENOENT
         {}
       end

data/lib/suspenders/adapters/heroku.rb

@@ -55,10 +55,6 @@ module Suspenders
         end
       end
 
-      def create_heroku_application_manifest_file
-        app_builder.template "app.json.erb", "app.json"
-      end
-
       def create_heroku_pipeline
         pipelines_plugin = `heroku help | grep pipelines`
         if pipelines_plugin.empty?
@@ -88,6 +84,19 @@ module Suspenders
         end
       end
 
+      def set_heroku_buildpacks
+        %w(staging production).each do |environment|
+          run_toolbelt_command(
+            "buildpacks:add --index 1 heroku/nodejs",
+            environment,
+          )
+          run_toolbelt_command(
+            "buildpacks:add --index 2 heroku/ruby",
+            environment,
+          )
+        end
+      end
+
       private
 
       attr_reader :app_builder
@@ -96,7 +105,7 @@ module Suspenders
         heroku_app_name = heroku_app_name_for(environment)
         <<~SHELL
 
-          if heroku join --app #{heroku_app_name} > /dev/null 2>&1; then
+          if heroku apps:info --app #{heroku_app_name} > /dev/null 2>&1; then
             git remote add #{environment} git@heroku.com:#{heroku_app_name}.git || true
             printf 'You are a collaborator on the "#{heroku_app_name}" Heroku app\n'
           else

data/lib/suspenders/app_builder.rb

@@ -7,7 +7,6 @@ module Suspenders
 
     def_delegators(
       :heroku_adapter,
-      :create_heroku_application_manifest_file,
       :create_heroku_pipeline,
       :create_production_heroku_app,
       :create_staging_heroku_app,
@@ -16,6 +15,7 @@ module Suspenders
       :set_heroku_honeybadger_env,
       :set_heroku_rails_secrets,
       :set_heroku_remotes,
+      :set_heroku_buildpacks,
     )
 
     def readme
@@ -97,18 +97,6 @@ module Suspenders
       copy_file "email.rb", "config/initializers/email.rb"
     end
 
-    def enable_rack_canonical_host
-      config = <<-RUBY
-  config.middleware.use Rack::CanonicalHost, ENV.fetch("APPLICATION_HOST")
-      RUBY
-
-      configure_environment "production", config
-    end
-
-    def enable_rack_deflater
-      configure_environment "production", "config.middleware.use Rack::Deflater"
-    end
-
     def setup_asset_host
       replace_in_file 'config/environments/production.rb',
         "# config.action_controller.asset_host = 'http://assets.example.com'",
@@ -186,101 +174,25 @@ config.public_file_server.headers = {
       )
     end
 
-    def replace_default_puma_configuration
-      copy_file "puma.rb", "config/puma.rb", force: true
-    end
-
-    def set_up_forego
-      copy_file "Procfile", "Procfile"
-    end
-
-    def setup_default_directories
-      [
-        'app/views/pages',
-        'spec/lib',
-        'spec/controllers',
-        'spec/helpers',
-        'spec/support/matchers',
-        'spec/support/mixins',
-        'spec/support/shared_examples'
-      ].each do |dir|
-        empty_directory_with_keep_file dir
-      end
-    end
-
-    def copy_dotfiles
-      directory("dotfiles", ".")
-    end
-
     def create_heroku_apps(flags)
       create_staging_heroku_app(flags)
       create_production_heroku_app(flags)
     end
 
-    def create_deploy_script
-      copy_file "bin_deploy", "bin/deploy"
-
-      instructions = <<-MARKDOWN
-
-## Deploying
-
-If you have previously run the `./bin/setup` script,
-you can deploy to staging and production with:
-
-    % ./bin/deploy staging
-    % ./bin/deploy production
-      MARKDOWN
-
-      append_file "README.md", instructions
-      run "chmod a+x bin/deploy"
-    end
-
     def configure_automatic_deployment
-      deploy_command = <<-YML.strip_heredoc
-      deployment:
-        staging:
-          branch: master
-          commands:
-            - bin/deploy staging
-      YML
-
-      append_file "circle.yml", deploy_command
+      append_file "Procfile", "release: bin/auto_migrate\n"
+      copy_file "bin_auto_migrate", "bin/auto_migrate"
     end
 
     def create_github_repo(repo_name)
       run "hub create #{repo_name}"
     end
 
-    def setup_bundler_audit
-      copy_file "bundler_audit.rake", "lib/tasks/bundler_audit.rake"
-      append_file "Rakefile", %{\ntask default: "bundle:audit"\n}
-    end
-
-    def setup_spring
-      bundle_command "exec spring binstub --all"
-    end
-
     def copy_miscellaneous_files
-      copy_file "browserslist", "browserslist"
       copy_file "errors.rb", "config/initializers/errors.rb"
       copy_file "json_encoding.rb", "config/initializers/json_encoding.rb"
     end
 
-    def customize_error_pages
-      meta_tags =<<-EOS
-  <meta charset="utf-8" />
-  <meta name="viewport" content="initial-scale=1" />
-      EOS
-
-      %w(500 404 422).each do |page|
-        path = "public/#{page}.html"
-        if File.exist?(path)
-          inject_into_file path, meta_tags, after: "<head>\n"
-          replace_in_file path, /<!--.+-->\n/, ''
-        end
-      end
-    end
-
     def remove_config_comment_lines
       config_files = [
         "application.rb",