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

Fix path to QueuesGettingStarted.java file #233

Merged
merged 1 commit into from Oct 1, 2018
Merged

Fix path to QueuesGettingStarted.java file #233

merged 1 commit into from Oct 1, 2018

Conversation

@c-w
Copy link
Contributor

@c-w c-w commented Oct 1, 2018

Description

The link to the QueuesGettingStarted.java file currently contains backslashes which causes a 404 on Github. Here is the link that causes the issue: https://github.com/Azure/azure-service-bus/blob/master/samples/Java/azure-servicebus/QueuesGettingStarted/.%5Csrc%5Cmain%5Cjava%5Ccom%5Cmicrosoft%5Cazure%5Cservicebus%5Csamples%5Cqueuesgettingstarted%5CQueuesGettingStarted.java

Replacing the backslashes with forward slashes as in this pull request fixes the problem.
This checklist is used to make sure that common guidelines for a pull request are followed.

  • I have read the contribution guidelines.
  • Title of the pull request is clear and informative.
  • If applicable, the code is properly documented.
  • The code builds without any errors.
The link to the QueuesGettingStarted.java file currently contains backslashes which causes a 404 on Github. See: https://github.com/Azure/azure-service-bus/blob/master/samples/Java/azure-servicebus/QueuesGettingStarted/.%5Csrc%5Cmain%5Cjava%5Ccom%5Cmicrosoft%5Cazure%5Cservicebus%5Csamples%5Cqueuesgettingstarted%5CQueuesGettingStarted.java

Replacing the backslashes with forward slashes as in this pull request fixes the problem.
@nemakam nemakam merged commit 3c897b1 into Azure:master Oct 1, 2018
1 check passed
@nemakam
Copy link
Contributor

@nemakam nemakam commented Oct 1, 2018

Thanks for this PR!

@c-w c-w deleted the patch-1 branch Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants