-
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
446 new ontohub oms generalization #1104
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} 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. [88/80]
please address the issues found by hound |
we need to get this long story to an end. Please solve all remaining points until Wednesday. The travis issue is caused by a wrong fixture and will be fixed by someone else quickly. |
7faba4e
to
03cf757
Compare
@@ -24,7 +24,7 @@ def show | |||
end | |||
end | |||
rescue Errno::ENOENT, NoMethodError => e | |||
redirect_to collection_path, flash: { error: "The file was not found: #{e.message}" } | |||
redirect_to collection_path, flash: {error: "The file was not found: #{e.message}"} |
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. [87/80]
- else | ||
= link_to "show only Logics with Ontologies", logics_path | ||
= link_to "show only logics with "+Settings.OMS.pluralize, logics_path |
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.
Please prefer String interpolation.
f7b0a64
to
16183b5
Compare
@@ -24,7 +24,10 @@ def show | |||
end | |||
end | |||
rescue Errno::ENOENT, NoMethodError => e | |||
redirect_to collection_path, flash: { error: "The file was not found: #{e.message}" } | |||
redirect_to collection_path | |||
,flash: { |
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.
unexpected token tCOMMA
16183b5
to
0a4c9ec
Compare
redirect_to collection_path, flash: { error: "The file was not found: #{e.message}" } | ||
redirect_to collection_path, | ||
flash: { | ||
error: "The file was not found: #{e.message}" |
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/TrailingComma: Put a comma after the last item of a multiline hash.
👍 |
446 new ontohub oms generalization
This is clean pull request without any mixed commit w.r.t indefinite article.