Paths

Table of Contentst

Differential D478

Style: better compliance with community Ruby coding style guide
ClosedPublic
Actions

Authored by dereckson on Jul 10 2016, 21:27.

Tags

None

Referenced Files

	F12373795: D478.diff
	Thu, Oct 30, 18:35

	F12368780: D478.id1168.diff
	Thu, Oct 30, 06:23

	Unknown Object (File)
	Sun, Oct 26, 17:40

	Unknown Object (File)
	Fri, Oct 24, 07:24

	Unknown Object (File)
	Tue, Oct 14, 03:30

	Unknown Object (File)
	Tue, Oct 14, 00:06

	Unknown Object (File)
	Sun, Oct 12, 13:18

	Unknown Object (File)
	Sun, Oct 12, 02:44

Subscribers

None

Details

Reviewers

Maniphest Tasks

T912: Follow the community Ruby Style Guide

Commits

rTOMMY228d017ece70: Style: better compliance with community Ruby coding style guide

Summary

The following points, detected by Rubocop, are improved:

if !something → unless something
Ruby 1.9 hash syntax is used
is_quality? → quality?
removed extraneous self

Test Plan

Run dashboard, ensure all cards are still available.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 1168.Jul 10 2016, 21:27

dereckson retitled this revision from to Style: better compliance with community Ruby coding style guide.

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

dereckson accepted this revision.Jul 10 2016, 21:28

dereckson edited edge metadata.

This revision is now accepted and ready to land.Jul 10 2016, 21:28

Closed by commit rTOMMY228d017ece70: Style: better compliance with community Ruby coding style guide (authored by dereckson, committed by dereckson). · Explain WhyJul 10 2016, 21:29

This revision was automatically updated to reflect the committed changes.

dereckson added a task: T912: Follow the community Ruby Style Guide.Jul 11 2016, 14:15

Revision Contents
Changeset List

Path

Size

34 lines

Diff 1169

tommy.rb

Loading...