Andy Waite
Andy Waite
e.g. #876 Other potential options: - display a warning in the UI - ignore `settings_data.json` by default, and have it be opt-in
In https://github.com/Shopify/themekit/pull/755 we changed to use unstable API so we could access the new checksum field. Now that this is being released in the 2020-10 update, we can switch back...
As discussed with @tanema, let's create a checklist of things to verify when making a release. For example: - Create a new theme - Download an existing theme - Deploy...
Due to some upcoming platform changes (Aug 25 email), we'll need to update the docs to indicate that the "Manage private apps" option will need to be enabled when setting...
To coincide with the new release, we should update the docs to cover how the behaviour of `deploy`, `download` and `watch` has changed.
This is specific to rspec-rails: The generator originally created a helper which required everything in `support/**/*.rb`. This was changed in https://github.com/rspec/rspec-rails/pull/1137 but I still find many projects which require all...
For example: ``` irb(main):026:0> "hello \##{1 + 1}" => "hello #2" ``` There's no need to escape the `#` here. This might be able to share some behaviour with `Style/RedundantRegexpEscape`....
(Apologies if this has been discussed previously, it's hard to search for with such a large issue volume.) ## Is your feature request related to a problem? Please describe. When...
Reading https://thoughtbot.com/blog/ruby-safe-navigation reminded me that over-use of `&.` is common. I think it would be helpful to add a guideline such as: > Avoid long chains of `&.`. Replace with...
A good argument here by @JoelQ: https://github.com/thoughtbot/til/blob/master/ruby/all-but-the-first-element-from-array.md https://ruby-doc.org/core-2.7.0/Array.html#method-i-drop