-
Notifications
You must be signed in to change notification settings - Fork 422
Fix broken url for testing instructions. #1208
Conversation
Thanks for the pull request, @RafayGhafoor! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
@robrap, Please review. |
Fixes broken url found in the docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
@robrap, I don't have the option to merge this PR, can you please do it. Thank you. |
@RafayGhafoor 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Thanks again @RafayGhafoor. I just needed to wait for tests to complete. Note that I squashed using the github UI, but otherwise that would be something you might need to do on other PRs. |
yes, I understand @robrap. I also made this PR from the UI and it wasn't allowing me to squash since I am not a maintainer I guess. |
I think you can only "Squash and Merge" through the UI. So otherwise, you'd need to get your fork locally and squash and push the changes back up. |
I've completed each of the following or determined they are not applicable: