Skip to content

docs: fix Samples repository path in README - #3169

Closed
FirmaSpring wants to merge 1 commit into
SixLabors:mainfrom
FirmaSpring:docs/fix-samples-path
Closed

docs: fix Samples repository path in README#3169
FirmaSpring wants to merge 1 commit into
SixLabors:mainfrom
FirmaSpring:docs/fix-samples-path

Conversation

@FirmaSpring

Copy link
Copy Markdown

Problem

The README samples link points at a path that no longer exists:

https://github.com/SixLabors/Samples/tree/main/ImageSharp

A GET with redirects returns 404. The samples repo now lives under SixLabors.Samples.ImageSharp.

Solution

Point the README at the current samples folder.

Verification

curl -sL -o /dev/null -w "%{http_code} %{url_effective}" -A "Mozilla/5.0" \
  "https://github.com/SixLabors/Samples/tree/main/ImageSharp"
# 404 https://github.com/SixLabors/Samples/tree/main/ImageSharp

curl -sL -o /dev/null -w "%{http_code} %{url_effective}" -A "Mozilla/5.0" \
  "https://github.com/SixLabors/Samples/tree/main/SixLabors.Samples.ImageSharp"
# 200 https://github.com/SixLabors/Samples/tree/main/SixLabors.Samples.ImageSharp

## Problem

The README samples link points at a path that no longer exists:

https://github.com/SixLabors/Samples/tree/main/ImageSharp

A GET with redirects returns 404. The samples repo now lives under `SixLabors.Samples.ImageSharp`.

## Solution

Point the README at the current samples folder.

## Verification

```
curl -sL -o /dev/null -w "%{http_code} %{url_effective}" -A "Mozilla/5.0" \
  "https://github.com/SixLabors/Samples/tree/main/ImageSharp"
# 404 https://github.com/SixLabors/Samples/tree/main/ImageSharp

curl -sL -o /dev/null -w "%{http_code} %{url_effective}" -A "Mozilla/5.0" \
  "https://github.com/SixLabors/Samples/tree/main/SixLabors.Samples.ImageSharp"
# 200 https://github.com/SixLabors/Samples/tree/main/SixLabors.Samples.ImageSharp
```

Co-authored-by: FirmamentalSpring <287222957+FirmaSpring@users.noreply.github.com>
@FirmaSpring

Copy link
Copy Markdown
Author

Closing as duplicate of #3168 (opened a minute earlier from the same account) which fixes the same Samples repository path. Sorry for the noise.

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.

1 participant