-
Notifications
You must be signed in to change notification settings - Fork 0
AYS-412 | Create Password Service Automation #184
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
Conversation
@agitrubard @Nbhtozcn selamlar, hata veren testler AYS-683 issue'su ile düzenlenen hata mesajından kaynaklı burada yapılan düzenlemelerle ilgili değildir. O issue test edilirken düzenlenmesi daha uygun olur. |
@cagla88 bu durumda o testleri disable etmemiz ve issue'yu not düşmemiz daha iyi olmaz mı? |
@agitrubard Bunun yerine iki issue'yu birlikte ilerletme kararı aldım zira tek bir kelime eklemek daha kolay bir yol. AYS-683 ile bu issue yu ilişkilendirip not düştüm |
@cagla88 eklemek daha kolay bir yol ama kolay yolu seçtiğimizde o issue Jira'da otomasyona yansıtılmamış gibi görünecek, bu durumda geri alıp farklı bir branch'te eklemeyi yapıp onu mergeleyip bunu sonra mergeleyebiliriz daha iyi olur 👍🏼 Görünmemesinin sebebi Squash and Merge yapıyor olmamız ve atılan commitlerin mergelendikten sonra pull request başlığına göre birleştiriliyor olması; |
This reverts commit 1a1af56.
Pull Request
Description:
Create Password positive and negative tests have been added.
Changes Made:
List the changes you made in this pull request.
Additional Information:
Add any additional information that might be relevant to reviewers.
Reviewer Instructions:
Provide instructions or guidelines for reviewers to follow when assessing your pull request.
Checklist
Before submitting your pull request, ensure the following:
Title and Branch Naming Conventions:
standard: Pull Request Naming Conventions
.
the Branch Naming Conventions
.
Local Testing:
Code Quality:
Documentation:
Reviewers and Assignees:
Labels and Associations:
Reviewer Instructions: