-
-
Notifications
You must be signed in to change notification settings - Fork 4
refactoring #53
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
base: master
Are you sure you want to change the base?
refactoring #53
Changes from all commits
ce69fc2
b829728
5442ea8
8c2a51a
72ceaba
642fce8
c9c9858
f6fd4c6
2f0ba5a
d14ea4c
d127b42
b2c16e4
e974ef5
1e70bca
b844f46
02d9e6c
da72a92
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -2,7 +2,7 @@ | |||||
|
|
||||||
| To report a potential vulnerability, follow the steps in **[Security](https://github.com/khulnaSoft/shipyard/blob/master/.github/SECURITY.md#reporting-a-security-issue)**. | ||||||
|
|
||||||
| For setup and usage guides, see **[ship.khulnasoft.com/docs](https://ship.khulnasoft.com/)** or the **[GitHub](https://github.com/khulnaSoft/shipyard)** repo. | ||||||
| For setup and usage guides, see **[live-ship.vercel.app/docs](https://live-ship.vercel.app/)** or the **[GitHub](https://github.com/khulnaSoft/shipyard)** repo. | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. issue (broader_impact): The documentation links are redirected from the repository's documentation host to the Vercel application host, but this Vue application has no Triggers: When users follow any of the updated documentation links. Suggested fix: Keep the documentation links on the domain serving the
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Fix the target of the documentation link. The link text says Proposed fix-For setup and usage guides, see **[live-ship.vercel.app/docs](https://live-ship.vercel.app/)** or the **[GitHub](https://github.com/khulnaSoft/shipyard)** repo.
+For setup and usage guides, see **[live-ship.vercel.app/docs](https://live-ship.vercel.app/docs)** or the **[GitHub](https://github.com/khulnaSoft/shipyard)** repo.📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
|
|
||||||
| To raise a bug, for something that's not working, **[Open a new Issue](https://github.com/khulnaSoft/shipyard/issues/new/choose)**. | ||||||
|
|
||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| ship.khulnasoft.com | ||
| live-ship.vercel.app | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🩺 Stability & Availability | 🟠 Major | ⚡ Quick win ✅ Runtime observed🏁 Script executed: #!/bin/bash
set -euo pipefail
rg -n 'CNAME|github-pages|gh-pages|vercel' .github package.json README.md
curl -fsSIL --max-time 15 https://live-ship.vercel.appRepository: khulnasoft/shipyard Length of output: 2846 🏁 Script executed: #!/bin/bash
set -euo pipefail
printf '%s\n' '--- CNAME ---'
cat -n CNAME
printf '%s\n' '--- documentation deployment workflow ---'
cat -n .github/workflows/build-docs-site.yml
printf '%s\n' '--- repository deployment/configuration references ---'
rg -n -i 'vercel|github pages|github-pages|custom domain|CNAME|deployment-branch|project' \
.github README.md package.json vercel.json vercel.* 2>/dev/null || true
printf '%s\n' '--- tracked deployment-related files ---'
git ls-files | rg -i '(^|/)(CNAME|vercel(\..*)?|.*pages.*|.*deploy.*)$' || trueRepository: khulnasoft/shipyard Length of output: 4895 Do not point the GitHub Pages The docs workflow publishes to 🤖 Prompt for AI Agents |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,7 +5,7 @@ | |
| <br/> | ||
| <img width="120" src="https://i.ibb.co/yhbt6CY/shipyard.png" /> | ||
| <br/> | ||
| <b><a href="./docs/showcase.md">User Showcase</a></b> | <b><a href="https://demo.ship.khulnasoft.com">Live Demo</a></b> | <b><a href="./docs/quick-start.md">Getting Started</a></b> | <b><a href="https://ship.khulnasoft.com/docs">Documentation</a></b> | <b><a href="https://github.com/khulnaSoft/shipyard">GitHub</a></b> | ||
| <b><a href="./docs/showcase.md">User Showcase</a></b> | <b><a href="https://demo.live-ship.vercel.app">Live Demo</a></b> | <b><a href="./docs/quick-start.md">Getting Started</a></b> | <b><a href="https://live-ship.vercel.app/docs">Documentation</a></b> | <b><a href="https://github.com/khulnaSoft/shipyard">GitHub</a></b> | ||
| </p> | ||
|
|
||
| <p align="center"> | ||
|
|
@@ -84,7 +84,7 @@ | |
|
|
||
| ## Demo ⚡ | ||
|
|
||
| **Live Instances**: [Demo 1](https://demo.ship.khulnasoft.com) (Live Demo) ┆ [Demo 2](https://live.ship.khulnasoft.com) (Shipyard Links) ┆ [Demo 3](https://dev.ship.khulnasoft.com) (Dev Preview) | ||
| **Live Instances**: [Demo 1](https://demo.live-ship.vercel.app) (Live Demo) ┆ [Demo 2](https://live-ship.vercel.app) (Shipyard Links) ┆ [Demo 3](https://dev.live-ship.vercel.app) (Dev Preview) | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win ✅ Runtime observed🏁 Script executed: #!/bin/bash
set -euo pipefail
for url in \
https://demo.live-ship.vercel.app \
https://live-ship.vercel.app \
https://dev.live-ship.vercel.app \
https://live-ship.vercel.app/docs
do
code="$(curl -L -sS -o /dev/null -w '%{http_code}' --max-time 15 "$url")"
case "$code" in
2??|3??) echo "$code $url" ;;
*) echo "unexpected HTTP status $code for $url" >&2; exit 1 ;;
esac
doneRepository: khulnasoft/shipyard Length of output: 194 🏁 Script executed: #!/bin/bash
set +e
printf '%s\n' '--- README context ---'
sed -n '1,12p;80,92p' README.md
printf '%s\n' '--- repository-owned deployment bindings ---'
find . -maxdepth 2 -type f \( -name 'vercel.json' -o -name 'package.json' -o -name 'next.config.*' -o -name 'netlify.toml' \) -print
printf '%s\n' '--- hostname probes ---'
for url in \
https://demo.live-ship.vercel.app \
https://live-ship.vercel.app \
https://dev.live-ship.vercel.app \
https://live-ship.vercel.app/docs
do
printf '%s: ' "$url"
curl -L -sS -o /dev/null -w 'http=%{http_code} remote=%{remote_ip} exit=%{exitcode}\n' \
--connect-timeout 5 --max-time 15 "$url"
doneRepository: khulnasoft/shipyard Length of output: 1963 🏁 Script executed: #!/bin/bash
set +e
printf '%s\n' '--- changed files ---'
git diff --name-only
printf '%s\n' '--- deployment configuration ---'
cat netlify.toml
printf '%s\n' '--- DNS resolution ---'
for host in demo.live-ship.vercel.app live-ship.vercel.app dev.live-ship.vercel.app; do
printf '%s: ' "$host"
getent ahosts "$host" 2>&1 | head -n 3
done
printf '%s\n' '--- direct HTTPS probes without proxy ---'
for url in \
https://demo.live-ship.vercel.app \
https://live-ship.vercel.app \
https://dev.live-ship.vercel.app \
https://live-ship.vercel.app/docs
do
printf '%s: ' "$url"
curl --noproxy '*' -L -sS -o /dev/null \
-w 'http=%{http_code} remote=%{remote_ip} exit=%{exitcode}\n' \
--connect-timeout 5 --max-time 15 "$url"
doneRepository: khulnasoft/shipyard Length of output: 2285 Fix the published demo links before merging. DNS resolves for all three hosts, but 🤖 Prompt for AI Agents |
||
|
|
||
| **Screenshots**: Checkout the [Showcase](./docs/showcase.md), to see example dashboards from the community | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick: The displayed setup-guide URL ends in
/docs, but the hyperlink target ishttps://live-ship.vercel.app/, so users clicking the documentation link are sent to the site root instead of the documentation page.Suggested fix: Change the hyperlink target to
https://live-ship.vercel.app/docsor make the displayed URL match the target.