Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

[WIP] Achivement #142

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

[WIP] Achivement #142

wants to merge 1 commit into from

Conversation

Raesta
Copy link
Collaborator

@Raesta Raesta commented Jul 9, 2021

No description provided.

@@ -0,0 +1,2 @@
class Achievement < ApplicationRecord
Copy link

Choose a reason for hiding this comment

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

[rubocop] reported by reviewdog 🐶
[Correctable] Style/FrozenStringLiteralComment: Missing frozen string literal comment.

@@ -0,0 +1,15 @@
class CreateAchievements < ActiveRecord::Migration[6.1]
Copy link

Choose a reason for hiding this comment

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

[rubocop] reported by reviewdog 🐶
[Correctable] Style/FrozenStringLiteralComment: Missing frozen string literal comment.

t.timestamps
end
end
def down
Copy link

Choose a reason for hiding this comment

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

[rubocop] reported by reviewdog 🐶
[Correctable] Layout/EmptyLineBetweenDefs: Expected 1 empty line between method definitions; found 0.

@@ -0,0 +1,7 @@
require "test_helper"
Copy link

Choose a reason for hiding this comment

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

[rubocop] reported by reviewdog 🐶
[Correctable] Style/FrozenStringLiteralComment: Missing frozen string literal comment.

@@ -0,0 +1,7 @@
require "test_helper"
Copy link

Choose a reason for hiding this comment

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

[rubocop] reported by reviewdog 🐶
[Correctable] Style/StringLiterals: Prefer single-quoted strings when you don't need string interpolation or special symbols.

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

Successfully merging this pull request may close these issues.

1 participant