Skip to content

Merge pull request #2 from JARVIS-discordbot/main#17

Merged
BenCos17 merged 30 commits into
mainfrom
Production
Jun 6, 2026
Merged

Merge pull request #2 from JARVIS-discordbot/main#17
BenCos17 merged 30 commits into
mainfrom
Production

Conversation

@BenCos17

@BenCos17 BenCos17 commented Jun 6, 2026

Copy link
Copy Markdown
Member

Create deploy.yml

Summary by CodeRabbit

Release Notes

  • Chores
    • Streamlined production deployment workflow with simplified branch configuration
    • Enhanced deployment mechanism with improved file transfer protocols for faster and more reliable production updates
    • Extended automation to include deployment of additional website resources

remember last tab for clusters
@BenCos17 BenCos17 merged commit 31c79f9 into main Jun 6, 2026
1 of 2 checks passed
@coderabbitai

coderabbitai Bot commented Jun 6, 2026

Copy link
Copy Markdown

Looking for one thing? Review this PR in Change Stack to search files, summaries, diffs, and code without losing your place.

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7342ef1d-b23d-4fd3-ad14-4e985b17d1c7

📥 Commits

Reviewing files that changed from the base of the PR and between 3cb6501 and 8f1b5ef.

📒 Files selected for processing (1)
  • .github/workflows/deploy.yml

📝 Walkthrough

Walkthrough

The GitHub Actions deployment workflow is updated to rename the job, restrict activation to the Production branch, and replace the scp-based file transfer with rsync commands that publish both index.html to the landing page and clusters.html to a new clusters directory.

Changes

Deployment Workflow Modernization

Layer / File(s) Summary
Workflow metadata and trigger configuration
.github/workflows/deploy.yml
Workflow name changed from "Deploy landing.html" to "Push to Production". The push trigger's branches filter is simplified to only Production.
Deployment steps: rsync publishing
.github/workflows/deploy.yml
Deployment steps replaced with rsync-based uploads: prior SSH connectivity test and scp transfer removed; index.html is now published to /var/www/jarvislandingpage/landing.html and a new step publishes clusters.html to /var/www/clusters/index.html.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 From scp to rsync, the pipeline takes flight,
Production branch glowing, deployments burn bright,
Two pages now dance—landing and clusters unite,
The workflow evolves, modernized right! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch Production

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@BenCos17 BenCos17 deleted the Production branch June 6, 2026 09:03
@BenCos17 BenCos17 restored the Production branch June 6, 2026 09:13
@BenCos17

BenCos17 commented Jun 6, 2026

Copy link
Copy Markdown
Member Author

accidentally deleted production....lol

BenCos17 added a commit that referenced this pull request Jun 6, 2026
Merge pull request #17 from JARVIS-discordbot/Production
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