Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add runner attribute to the message initialization #126

Merged
merged 1 commit into from
Feb 16, 2016

Conversation

skateman
Copy link
Contributor

@skateman skateman commented Feb 4, 2016

Resolves #113

Message.new(path: 'path/to',
line: line,
level: :error,
msg: 'Line Error')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use this style of alignment (consistent in project, easier to change, does not require editor plugins 😄):

Message.new(path: 'path/to',
            line: line,
            level: :error,
            msg: 'Line Error')

@skateman
Copy link
Contributor Author

@mmozuras all fixed, now it's a quite small patch :)

mmozuras added a commit that referenced this pull request Feb 16, 2016
Add runner attribute to the message initialization
@mmozuras mmozuras merged commit 38ea17b into prontolabs:master Feb 16, 2016
@mmozuras
Copy link
Member

@skateman thanks! 😄

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants