Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

Tweak Detekt missing exception message #53

Merged
merged 3 commits into from Jan 31, 2018

Conversation

rock3r
Copy link
Contributor

@rock3r rock3r commented Jan 30, 2018

This PR tweaks the Detekt missing error message. Just adding a missing full stop and a new line for readability.

@@ -11,7 +11,7 @@ import org.gradle.api.Task
class DetektConfigurator implements Configurator {

private static final String DETEKT_PLUGIN = 'io.gitlab.arturbosch.detekt'
private static final String DETEKT_NOT_APPLIED = 'The detekt plugin is configured but not applied. Please apply the plugin in your build script For more information see https://github.com/arturbosch/detekt.'
private static final String DETEKT_NOT_APPLIED = 'The detekt plugin is configured but not applied. Please apply the plugin in your build script.\nFor more informations see https://github.com/arturbosch/detekt.'
Copy link
Contributor

Choose a reason for hiding this comment

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

pretty sure information doesn't need that s as it's considered uncountable 😅 It would be good to ask some English speaker tho 🇬🇧 🇺🇸 🇨🇦

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looks like you're right, will tweak in the morning :)

Copy link
Contributor

@tobiasheine tobiasheine left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@tasomaniac tasomaniac merged commit 4160752 into PT-491/integrate_detekt Jan 31, 2018
@tasomaniac tasomaniac deleted the tweak-detekt-exception-message branch January 31, 2018 08:49
@rock3r
Copy link
Contributor Author

rock3r commented Jan 31, 2018

@tasomaniac there was a :x: do not merge label for a reason... I was working on this 😓

Oh well I'll have to open a new PR.

@tasomaniac
Copy link
Contributor

@rock3r sorry about that :) Didn't see it.

@tasomaniac
Copy link
Contributor

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants