Repository navigation
Release v3.7.9 #1917
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
Release v3.7.9 #1917
Changes from 26 commits
5aff529
c499293
f64b209
fe25fdc
a2d0db2
0f35c14
aa40bc3
743be29
970f6a3
a7233d1
dde7428
6e38bd5
d8baf44
6424c82
37a4a66
2bd8e05
6b7f331
879eda9
53fd36e
d7a14ce
dc8b856
9dc377e
1d38900
0e8148d
9e1d488
fb2fb7c
b8746b1
0c27b89
b73f361
d3c1abc
f6fdfea
88972bf
1a342ae
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 | ||||
|---|---|---|---|---|---|---|
|
|
@@ -22,7 +22,7 @@ Deploy OmniRoute to the production VPSs using `npm pack + scp` + PM2. | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router && rm -f omniroute-*.tgz && rm -rf .next/cache app/.next/cache && npm run build:cli && rm -rf app/logs app/coverage app/.git app/.app-build-backup* && npm pack --ignore-scripts | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute && rm -f omniroute-*.tgz && rm -rf .next/cache app/.next/cache && npm run build:cli && rm -rf app/logs app/coverage app/.git app/.app-build-backup* && npm pack --ignore-scripts | ||||||
|
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. The deployment script uses a hardcoded, user-specific absolute path (
Suggested change
|
||||||
| ``` | ||||||
|
|
||||||
| ### 2. Copy to both VPS and install | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -17,7 +17,7 @@ Deploy OmniRoute to the Local VPS using `npm pack + scp` + PM2. | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router && rm -f omniroute-*.tgz && rm -rf .next/cache app/.next/cache && npm run build:cli && rm -rf app/logs app/coverage app/.git app/.app-build-backup* && npm pack --ignore-scripts | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute && rm -f omniroute-*.tgz && rm -rf .next/cache app/.next/cache && npm run build:cli && rm -rf app/logs app/coverage app/.git app/.app-build-backup* && npm pack --ignore-scripts | ||||||
|
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. The deployment script uses a hardcoded, user-specific absolute path (
Suggested change
|
||||||
| ``` | ||||||
|
|
||||||
| ### 2. Copy to Local VPS and install | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -226,7 +226,7 @@ git checkout main | |||||
| git pull origin main | ||||||
|
|
||||||
| # Build and pack locally | ||||||
| cd /home/diegosouzapw/dev/proxys/9router && rm -f omniroute-*.tgz && rm -rf .next/cache app/.next/cache && npm run build:cli && rm -rf app/logs app/coverage app/.git app/.app-build-backup* && npm pack --ignore-scripts | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute && rm -f omniroute-*.tgz && rm -rf .next/cache app/.next/cache && npm run build:cli && rm -rf app/logs app/coverage app/.git app/.app-build-backup* && npm pack --ignore-scripts | ||||||
|
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. The deployment script uses a hardcoded, user-specific absolute path (
Suggested change
|
||||||
|
|
||||||
| # Deploy to LOCAL VPS (192.168.0.15) | ||||||
| scp omniroute-*.tgz root@192.168.0.15:/tmp/ | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -20,7 +20,7 @@ Automatically bump the project version, generate CHANGELOG entries from git hist | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
|
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. The script uses a hardcoded, user-specific absolute path (
Suggested change
|
||||||
| CURRENT_VERSION=$(node -p "require('./package.json').version") | ||||||
| LAST_TAG=$(git describe --tags --abbrev=0 2>/dev/null || echo "") | ||||||
| CURRENT_BRANCH=$(git branch --show-current) | ||||||
|
|
@@ -64,7 +64,7 @@ npm version "$VERSION" --no-git-tag-version | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
| LAST_TAG=$(git describe --tags --abbrev=0 2>/dev/null) | ||||||
| echo "=== Commits since $LAST_TAG ===" | ||||||
| git log "$LAST_TAG"..HEAD --pretty=format:"%h %s" --no-merges | head -100 | ||||||
|
|
@@ -133,7 +133,7 @@ The date must be today's date in `YYYY-MM-DD` format. | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
| VERSION=$(node -p "require('./package.json').version") | ||||||
|
|
||||||
| # Update docs/openapi.yaml version | ||||||
|
|
@@ -156,7 +156,7 @@ echo "✓ All workspace packages synced to $VERSION" | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
| VERSION=$(node -p "require('./package.json').version") | ||||||
| OLD_VERSION_PATTERN='[0-9]\+\.[0-9]\+\.[0-9]\+' | ||||||
|
|
||||||
|
|
@@ -174,7 +174,7 @@ echo "✓ llm.txt → $VERSION" | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
| npm install | ||||||
| echo "✓ Lock file regenerated" | ||||||
| ``` | ||||||
|
|
@@ -236,7 +236,7 @@ For each file in `docs/` (excluding `docs/i18n/`), review if CHANGELOG changes a | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
| npm run lint | ||||||
| ``` | ||||||
|
|
||||||
|
|
@@ -245,7 +245,7 @@ npm run lint | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
| npm test | ||||||
| ``` | ||||||
|
|
||||||
|
|
@@ -254,7 +254,7 @@ npm test | |||||
| // turbo | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
| VERSION=$(node -p "require('./package.json').version") | ||||||
| echo "Expected version: $VERSION" | ||||||
| echo "" | ||||||
|
|
@@ -299,7 +299,7 @@ grep "^## \[" CHANGELOG.md | head -2 | |||||
| // turbo-all | ||||||
|
|
||||||
| ```bash | ||||||
| cd /home/diegosouzapw/dev/proxys/9router | ||||||
| cd /home/diegosouzapw/dev/proxys/OmniRoute | ||||||
| git add -A | ||||||
| VERSION=$(node -p "require('./package.json').version") | ||||||
| git commit -m "chore(release): bump to v$VERSION — changelog, docs, version sync" | ||||||
|
|
||||||
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.
The deployment script uses a hardcoded, user-specific absolute path (
/home/diegosouzapw/dev/proxys/OmniRoute). This makes the script non-portable and difficult for other developers to use. It's better to use a relative path or a command to dynamically find the project root.