-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
1087 correct indefinite article for oms #1111
Conversation
@@ -75,7 +75,7 @@ def show | |||
|
|||
def destroy | |||
if resource.is_imported? | |||
flash[:error] = "Can't delete an ontology that is imported by another one." | |||
flash[:error] = "Can't delete an #{Settings.OMS.with_indefinite_article} that is imported by another one." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Metrics/LineLength: Line is too long. [112/80]
0254b5c
to
bbfbfcc
Compare
@@ -14,7 +14,8 @@ def destroy | |||
super | |||
rescue StandardError => e | |||
unmark_as_destroying | |||
raise e.class, "Can't delete repository: It contains an ontology that is imported by another repository." | |||
raise e.class, "Can't delete repository: It contains \ | |||
#{Settings.OMS.with_indefinite_article} that is imported by another repository." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Metrics/LineLength: Line is too long. [84/80]
bbfbfcc
to
64b41cb
Compare
@@ -14,7 +14,8 @@ def destroy | |||
super | |||
rescue StandardError => e | |||
unmark_as_destroying | |||
raise e.class, "Can't delete repository: It contains an #{Settings.OMS} that is imported by another repository." | |||
raise e.class, "Can't delete repository: It contains |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Style/TrailingWhitespace: Trailing whitespace detected.
64b41cb
to
52e5140
Compare
52e5140
to
d50b235
Compare
d50b235
to
b14a891
Compare
I just rebased staging. In my opinion, it's a 👍 |
…_for_OMS 1087 correct indefinite article for oms
added new gem and used its method 'with_indefinite_article' for all the found instances where it can be applied.