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.

Sign up for GitHub

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

Jump to bottom

SPT-1990 ISG Анализ версий SPM-пакетов #1

Merged
merged 3 commits into from
Apr 16, 2024
Merged

SPT-1990 ISG Анализ версий SPM-пакетов #1

merged 3 commits into from
Apr 16, 2024

Conversation

Goldilocks97
Copy link

Что сделано

  • Форкнут плагин
  • Добавлена проверка зависимостей локальных пакетов
  • Откатился на версию 0.1.0, так как текущая версия оригинального плагина не поддерживает версию ruby 2.7.6

Как тестировать

  • Создана ветка для теста, в ней уже добавлены все поддерживаемые версии зависимостей для теста. Чтобы проверить свои случаи, внесите правки на этой ветки (в том числе можно попробовать настроить Dangerfile), сделайте push, дождитесь результата

@Goldilocks97 Goldilocks97 self-assigned this Mar 25, 2024
NullIsOne
NullIsOne requested changes Mar 25, 2024
View reviewed changes
Copy link

@NullIsOne NullIsOne left a comment

Choose a reason for hiding this comment

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

Также вижу конфликт с main веткой в plugin.rb. Надо его разрешить.

lib/spm_version_updates/plugin.rb Outdated Show resolved Hide resolved
lib/spm_version_updates/local.rb Show resolved Hide resolved
@Goldilocks97
Copy link
Author

  • Добавил тесты на отслеживание локальных зависимостей
  • Добавил тесты на парсинг локальных манифестов
  • Отформатировал код (rubycop)

@Goldilocks97 Goldilocks97 force-pushed the SPT-1990-local-package-dependencies branch from 68f1b1d to cb4f6f1 Compare March 27, 2024 09:58
@Goldilocks97 Goldilocks97 changed the base branch from main to develop March 27, 2024 10:18
@Goldilocks97
Copy link
Author

Изменил base, временно будем использовать ветку develop, пока не перейдем на руби 3+

@Goldilocks97 Goldilocks97 requested a review from NullIsOne March 27, 2024 10:20
chausovSurfStudio
chausovSurfStudio reviewed Apr 15, 2024
View reviewed changes
Copy link

@chausovSurfStudio chausovSurfStudio left a comment

Choose a reason for hiding this comment

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

честно признаюсь - тесты не осилил, надеюсь на Никиту)
поглядел вместо этого вот этот ПР - есть пару вопросов. Давай порешаем - и пойдем тестить эту тему!

spec/spm_version_updates_spec.rb Show resolved Hide resolved
@chausovSurfStudio chausovSurfStudio merged commit c8c7bbc into develop Apr 16, 2024
@chausovSurfStudio chausovSurfStudio deleted the SPT-1990-local-package-dependencies branch April 16, 2024 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NullIsOne NullIsOne NullIsOne approved these changes

@chausovSurfStudio chausovSurfStudio chausovSurfStudio approved these changes

@mrandrewsmith mrandrewsmith Awaiting requested review from mrandrewsmith

Assignees

@Goldilocks97 Goldilocks97

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Goldilocks97 @chausovSurfStudio @NullIsOne