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

swift: update documentation link #13629

Closed

Conversation

nelsonfigueroa
Copy link
Collaborator

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page(s) have at most 8 examples.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The PR title conforms to the recommended templates.

@github-actions github-actions bot added the page edit Changes to an existing page(s). label Sep 8, 2024
@sebastiaanspeck
Copy link
Member

https://www.swift.org and https://swift.org are the same website. Can you clarify why you want to prefix it with www?

Copy link
Member

@spageektti spageektti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are removing www from links, not adding (#13633), so If www* redirects to the same site we should not add that.

@nelsonfigueroa
Copy link
Collaborator Author

My reasoning was that it's better to have the direct URL to documentation rather than a URL that redirects, as redirects may not be in place forever. I will leave it up to the maintainers though.

@spageektti
Copy link
Member

My reasoning was that it's better to have the direct URL to documentation rather than a URL that redirects, as redirects may not be in place forever. I will leave it up to the maintainers though.

I agree with you for other redirects, but I can't agree that it matters for this kind of redirect.

@nelsonfigueroa
Copy link
Collaborator Author

Fair enough, closing this.

@nelsonfigueroa nelsonfigueroa deleted the swift-link-update branch September 8, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
page edit Changes to an existing page(s).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants