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.

Sign up for GitHub

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

Jump to bottom

[docs] absolute assets links #7049

Merged
merged 3 commits into from
Dec 24, 2021
Merged

[docs] absolute assets links #7049

merged 3 commits into from
Dec 24, 2021

Conversation

ignatiusmb
Copy link
Member

Final remaining relative urls that hasn't been converted, looks like I missed checking some reference to the static assets.

Closes #7047

Before submitting the PR, please make sure you do the following

  • It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
  • Prefix your PR title with [feat], [fix], [chore], or [docs].
  • This message body should clearly illustrate what problems it solves.
  • Ideally, include a test that fails without this PR but passes with it.

Tests

  • Run the tests with npm test and lint the project with npm run lint

@bluwy bluwy mentioned this pull request Dec 24, 2021
Closed
@ignatiusmb ignatiusmb mentioned this pull request Dec 24, 2021
Closed
@benmccann benmccann merged commit 7db797e into sveltejs:master Dec 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bluwy bluwy bluwy approved these changes

@dummdidumm dummdidumm dummdidumm approved these changes

@antony antony antony approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ignatiusmb @antony @dummdidumm @bluwy @benmccann