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

Рефакторинг. Часть 3 #691

Merged
merged 14 commits into from
Mar 5, 2020
Merged

Рефакторинг. Часть 3 #691

merged 14 commits into from
Mar 5, 2020

Conversation

theshadowco
Copy link

Рефакторинг управляемой формы

  • Снижение сложности и удаление ненужного кода
  • Каноническое написание
  • снижение дублирования

@bia-tech
Рефакторинг
9c667c6 
 - пробелы
 - сложности
 - необязательные до обязательных
 - оптимизация записи настроек
@bia-tech
Рефакторинг
5e4b77a 
 - кэширование мигнорируемых видов элементов
 - снижение сложностей методов
 - снижение копипаста
 - каноническое написание
@bia-tech
Снижение сложности и удаление ненужного кода
48724d4
@bia-tech
Снижение сложности и удаление ненужного кода
6dbb865
@bia-tech
Merge remote-tracking branch 'vendor/develop' into feature/p3
44a0047 
# Conflicts:
#	VanessaAutomation/Forms/УправляемаяФорма/Ext/Form/Module.bsl
@theshadowco
Copy link
Author

run tests

@theshadowco
Copy link
Author

run tests

1 similar comment
@theshadowco
Copy link
Author

run tests

@ViktorErmakov
Copy link
Contributor

Валерий, а Вы рефакторинг делаете на основании каких проверок, Sonar или АПК или вручную?
Спасибо!

@theshadowco
Copy link
Author

Валерий, а Вы рефакторинг делаете на основании каких проверок, Sonar или АПК или вручную?
Спасибо!

Bslls показывает где болит, я помогаю в снижении боли

@ViktorErmakov
Copy link
Contributor

Валерий, а Вы рефакторинг делаете на основании каких проверок, Sonar или АПК или вручную?
Спасибо!

Bslls показывает где болит, я помогаю в снижении боли

т.е. в VSC просматриваете код?
Спасибо за ответы

@theshadowco
Copy link
Author

Валерий, а Вы рефакторинг делаете на основании каких проверок, Sonar или АПК или вручную?
Спасибо!

Bslls показывает где болит, я помогаю в снижении боли

т.е. в VSC просматриваете код?
Спасибо за ответы

да, в нем

@theshadowco
Copy link
Author

либо в EDT, от настроения зависит

@theshadowco
Copy link
Author

run tests

@Pr-Mex
Copy link
Owner

Pr-Mex commented Feb 29, 2020

Run tests

@theshadowco
Copy link
Author

@Pr-Mex
Это норальные ошибки или я что-то сломал?

@theshadowco
Copy link
Author

run tests

1 similar comment
@Pr-Mex
Copy link
Owner

Pr-Mex commented Mar 2, 2020

run tests

@theshadowco
Copy link
Author

run tests

@theshadowco
Copy link
Author

run tests

2 similar comments
@theshadowco
Copy link
Author

run tests

@Pr-Mex
Copy link
Owner

Pr-Mex commented Mar 3, 2020

run tests

@Pr-Mex
Copy link
Owner

Pr-Mex commented Mar 5, 2020

Спасибо!

@Pr-Mex Pr-Mex merged commit 1dce8ee into Pr-Mex:develop Mar 5, 2020
@theshadowco theshadowco deleted the feature/p3 branch March 6, 2020 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@theshadowco @ViktorErmakov @Pr-Mex @bia-tech