Skip to content
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

install annotate gem #725

Merged
merged 2 commits into from
Feb 22, 2024
Merged

install annotate gem #725

merged 2 commits into from
Feb 22, 2024

Conversation

liz4chernyshova
Copy link
Contributor

Устанавливаем gem annotate

Gemfile Outdated
@@ -78,6 +78,7 @@ gem 'wkhtmltopdf-binary'

group :development, :staging, :test do
gem 'faker'
gem 'annotate'
Copy link
Contributor

Choose a reason for hiding this comment

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

скорее всего должна быть development и test

Copy link
Contributor

Choose a reason for hiding this comment

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

в целом и для тестов это тоже не надо, только development

@@ -73,6 +73,9 @@ lint-style:
linter-code-fix:
bundle exec rubocop -A

editor-setup:
Copy link
Contributor

Choose a reason for hiding this comment

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

команда наверное не нужна

@amshkv
Copy link
Contributor

amshkv commented Feb 20, 2024

и ты саму команду не запустила

Gemfile Outdated
@@ -78,6 +78,7 @@ gem 'wkhtmltopdf-binary'

group :development, :staging, :test do
gem 'faker'
gem 'annotate'
Copy link
Contributor

Choose a reason for hiding this comment

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

в целом и для тестов это тоже не надо, только development

@usernaimandrey
Copy link
Contributor

ошибку линтера надо поправить и команду запустит

@usernaimandrey usernaimandrey marked this pull request as ready for review February 22, 2024 19:56
@usernaimandrey usernaimandrey merged commit cef8f47 into main Feb 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants