further improve the dockerfile #34
Annotations
10 errors and 2 warnings
test:
app/channels/application_cable/channel.rb#L4
Style/RedundantSelf: Redundant `self` detected.
|
test:
app/controllers/finance/bank_gateways_controller.rb#L12
Style/RescueStandardError: Avoid rescuing without specifying an error class.
|
test:
app/controllers/finance/bank_gateways_controller.rb#L12
Naming/RescuedExceptionsVariableName: Use `e` instead of `error`.
|
test:
app/controllers/finance/bank_gateways_controller.rb#L25
Style/RescueStandardError: Avoid rescuing without specifying an error class.
|
test:
app/controllers/finance/bank_gateways_controller.rb#L25
Naming/RescuedExceptionsVariableName: Use `e` instead of `error`.
|
test:
app/controllers/finance/financial_transactions_controller.rb#L64
Naming/RescuedExceptionsVariableName: Use `e` instead of `error`.
|
test:
app/controllers/finance/financial_transactions_controller.rb#L70
Style/HashSyntax: Use the new Ruby 1.9 hash syntax.
|
test:
app/controllers/finance/invoices_controller.rb#L91
Style/IfUnlessModifier: Favor modifier `unless` usage when having a single-line body. Another good alternative is the usage of control flow `&&`/`||`.
|
test:
app/controllers/finance/invoices_controller.rb#L102
Style/IfUnlessModifier: Favor modifier `unless` usage when having a single-line body. Another good alternative is the usage of control flow `&&`/`||`.
|
test:
app/controllers/self_service_controller.rb#L28
Style/GuardClause: Use a guard clause (`return unless @ordergroup.nil?`) instead of wrapping the code inside a conditional expression.
|
test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|