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

ELY-2638 Simplify test assertions in SSLAuthenticationTest #1959

Conversation

nidhiazad
Copy link

@Skyllarr
Copy link
Contributor

Hello @nidhiazad , please make sure there is 1 commit for 1 issue. Currently, there is 1 commit with the commit message "Your commit message here" , so please update it. Please also, create a jira acount and assign the issue to you, so other participants know that it was taken. Thank you!

@nidhiazad
Copy link
Author

nidhiazad commented Sep 22, 2023 via email

@nidhiazad
Copy link
Author

nidhiazad commented Sep 22, 2023 via email

@Skyllarr
Copy link
Contributor

@nidhiazad I will explain this on the preseentation

@Skyllarr
Copy link
Contributor

Hello @nidhiazad , just a minor, please make sure that there is 1 commit for 1 issue. Thank you!

@nidhiazad
Copy link
Author

sure I am following this practice of creating 1 branch , 1PR for 1 issue

@PrarthonaPaul
Copy link
Contributor

sure I am following this practice of creating 1 branch , 1PR for 1 issue

Since this issue has another PR, which you are working on, you can go ahead and close this one when you can. Thanks!

@nidhiazad
Copy link
Author

Closing this PR. Thanks

@nidhiazad nidhiazad closed this Sep 22, 2023
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.

3 participants