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

Перенос квик-фиксов по проверке путей данных #14

Merged
merged 14 commits into from Jan 12, 2022

Conversation

marmyshev
Copy link
Collaborator

@marmyshev marmyshev commented Jan 12, 2022

Что сделано

Добавлены новые квик-фиксы:

  • удаление битой ссылки в пути к данным
  • удаление элемента формы с битой ссылкой

Чек-лист

Общее:

  • ветка PR обновлена из master и нет конфликтов
  • Тесты-кейсы, JUnit тесты правильного и неправильного состояния
  • Измененные Вами исходники отформатированы в соответствии с конвенцией
  • Авто-аудит (SonarQube и CheckStyle) пройден, покрытие кода хорошее, ошибок нет, плохой код устранен
  • Добавлена запись в ИСТОРИЮ ИЗМЕНЕНИЯ, включаемая в пользовательскую документацию плагина

Если применимо:

  • Пользовательская документация на доп.инструменты написана (на русском)
  • Описание проверок - на двух языках

Закрываемые задачи

Closes #12

- удаление битой ссылки в пути к данным
- удаление элемента формы с битой ссылкой
@marmyshev marmyshev added this to the 0.1 - EDT 2021.3 milestone Jan 12, 2022
@marmyshev marmyshev added Analyze Run SonarQube analysis on PR from forks and removed Analyze Run SonarQube analysis on PR from forks labels Jan 12, 2022
@sonarcloud
Copy link

sonarcloud bot commented Jan 12, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

30.0% 30.0% Coverage
0.0% 0.0% Duplication

@marmyshev marmyshev merged commit 2b9842b into release/0.1.0 Jan 12, 2022
@marmyshev marmyshev deleted the feature/12-move-data-path-fix branch January 12, 2022 20:18
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

1 participant