trotzig / diffux / 231

Builds Branch Commit Type Ran Committer Via Coverage
231 master Remove unused StaticPagesHelper and UrlsHelper These files were likely generated by some scaffolding process, but were never fleshed out. Removing them to help streamline the codebase. Change-Id: Ib129e2d74 push lencioni travis-ci pending completion   set done
230 master Use the new "sexy" validations for Project As pointed out by Rubocop. This resolves all of the cops for this file. Change-Id: I9632bea79 push lencioni travis-ci pending completion   set done
229 master Use the new "sexy" validations for Snapshot As pointed out by Rubocop. This resolves all of the cops for this file. Change-Id: Ifb56f96f3 push lencioni travis-ci pending completion   set done
228 master Use the new "sexy" validations for SnapshotDiff As pointed out by Rubocop. This resolves all of the cops for this file. Change-Id: Id82cf7a6b push lencioni travis-ci pending completion   set done
227 master Use the new "sexy" validations for Sweep model As pointed out by Rubocop. This resolves all of the cops for this file. Change-Id: Ifb063928e push lencioni travis-ci pending completion   set done
226 master Use the new "sexy" validations in Viewport As pointed out by Rubocop. Also, this commit adds a top-level class documentation comment to this class. Change-Id: I4e4cea055 push lencioni travis-ci pending completion   set done
221 master Replace `Proc.new` with `proc` This is another thing pointed out by rubocop. Change-Id: I3d283edb3 push lencioni travis-ci pending completion   set done
220 master Squelch a bunch of rubocop lints This cuts down on the noise and increases our confidence in our tools. Change-Id: Ia55ee4519 push lencioni travis-ci pending completion   set done
219 master Add missing `should` in SnapshotsController spec Change-Id: Ic6927213b push lencioni travis-ci pending completion   set done
218 master Add rubocop config This configuration file should help us configure rubocop so it behaves more like we'd like it to. I started with a few options that jumped out at me, but I expect that this will evolve over time. Change-Id: I4fdc6081ed8a43038e... push lencioni travis-ci pending completion   set done
217 master Fix YARD @param format We had some inconsistencies with how these YARD tags are supposed to be formatted. This commit attempts to resolve them. Change-Id: I4772ac42a push lencioni travis-ci pending completion   set done
216 master Make status block rows keyboard-focusable Now that we have keyboard navigation for the diff clusters, it helps to be able to navigate through the status block table rows as well. push trotzig travis-ci pending completion   set done
215 master Refactor breadcrumbs into partial We had a lot of duplicated markup related to breadcrumbs so I refactored that code to render through a partial instead. This sets us up for some improvements to keyboard shortcuts, I have a goal of adding somethi... push trotzig travis-ci pending completion   set done
214 master Change keyboard shortcuts for `[` and `]` These keys used to move up/down once and then auto-click the currently focused link. Now that we've moved to pager ("Previous"/"Next") buttons instead of the "In review" panel, these shortcuts didn't real... push trotzig travis-ci pending completion   set done
213 master Make diff clusters keyboard-focusable Now that we have information about where all the diff clusters are, we can use it to enhance the snapshot review process. I've added each cluster to the diff image container and made them keyboard-focusable. ... push trotzig travis-ci pending completion   set done