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

API docs: remove an absolute URL #2364

Merged
merged 1 commit into from
Apr 6, 2019
Merged

API docs: remove an absolute URL #2364

merged 1 commit into from
Apr 6, 2019

Conversation

mindrones
Copy link
Member

I originally started this PR because the link [stores](tutorial/writable-stores)
renders as http://localhost:3000/tutorial/writable-stores on https://v3.svelte.technology/docs#svelte-store.
No idea why because the rest of the relative links in content/ have the same format.

I originally started this PR because the link `[stores](tutorial/writable-stores)`
renders as http://localhost:3000/tutorial/writable-stores on https://v3.svelte.technology/docs#svelte-store.
No idea why because the rest of the relative links in content/ have the same format.
@codecov-io
Copy link

Codecov Report

Merging #2364 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2364   +/-   ##
=======================================
  Coverage   91.83%   91.83%           
=======================================
  Files           1        1           
  Lines          49       49           
=======================================
  Hits           45       45           
  Misses          4        4

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c853e44...f6b399f. Read the comment docs.

@Conduitry
Copy link
Member

There was a PR #2352 that was opened for that which I merged but which probably just hasn't been deployed.

Removing this other absolute URL does also make sense though.

@Rich-Harris Rich-Harris merged commit d73c833 into sveltejs:master Apr 6, 2019
@mindrones mindrones deleted the site-apidoc-fix branch April 6, 2019 14:55
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.

4 participants