Skip to content

docs: use https links#2518

Merged
justin808 merged 1 commit intoshakacode:masterfrom
GeObts:docs/https-links-readme-projects
Mar 5, 2026
Merged

docs: use https links#2518
justin808 merged 1 commit intoshakacode:masterfrom
GeObts:docs/https-links-readme-projects

Conversation

@GeObts
Copy link
Copy Markdown
Contributor

@GeObts GeObts commented Mar 4, 2026

  • Switch HTTP links to HTTPS in README.md and PROJECTS.md
  • Each link verified to resolve over HTTPS
  • Updated links: shakacode.com, madroneco.com, railsonmaui.com, foxford.ru

Summary by CodeRabbit

  • Documentation
    • Updated all external project reference links to use secure HTTPS protocol
    • Added two new organizations (LocalWise and Ellevest) to the Commercial Products Live section
    • Revised project descriptions and formatting across multiple entries to improve consistency
    • Updated the career page link to HTTPS

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c22af122-88be-431c-9f32-8a5762ca05b3

📥 Commits

Reviewing files that changed from the base of the PR and between 2e5dbbc and 68eb9a4.

📒 Files selected for processing (2)
  • PROJECTS.md
  • README.md

Walkthrough

Documentation files updated with HTTPS URL conversions for improved security across multiple product links and career page. Two new commercial products added to the projects list with descriptions.

Changes

Cohort / File(s) Summary
Documentation URL Updates & Product Additions
PROJECTS.md, README.md
Converted multiple HTTP links to HTTPS for security compliance (Hawaii Chee, GuavaPass, Foxford.ru, Madrone Analytics, Blink Inc, ShakaCode career URL). Added new commercial products LocalWise and Ellevest to the projects list. Updated descriptions for consistency.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 With whiskers twitching, we hop with glee,
HTTP → HTTPS, secure as can be!
LocalWise and Ellevest join the crew,
Documentation polished, safe and true! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'docs: use https links' accurately summarizes the main change in the pull request - converting HTTP links to HTTPS in documentation files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Mar 4, 2026

Greptile Summary

This PR is a documentation-only change that upgrades insecure http:// links to https:// across PROJECTS.md and README.md, improving security posture for links to external sites.

  • Updates 6 HTTP links in PROJECTS.md covering shakacode.com (×4), madroneco.com, railsonmaui.com, and foxford.ru
  • Updates 1 HTTP link in README.md for shakacode.com/career
  • All changes have been verified by the author to resolve over HTTPS
  • One HTTP link for http://www.pivotaltracker.com/ on line 24 of PROJECTS.md was missed and remains as an inconsistency

Confidence Score: 4/5

  • Safe to merge with a minor fix — this PR contains only documentation link upgrades with no code changes or runtime risk.
  • The PR is fundamentally sound with straightforward HTTP → HTTPS link substitutions. The one missed link (Pivotal Tracker on line 24 of PROJECTS.md) is a minor inconsistency that should be corrected before merging to maintain consistency with the PR's stated goal of upgrading all HTTP links to HTTPS.
  • PROJECTS.md requires updating the Pivotal Tracker link on line 24 from http:// to https://.

Last reviewed commit: 68eb9a4

Comment thread PROJECTS.md
- **[HVMN](https://hvmn.com)**: Web ecommerce site for "biohacking" products.
- **[GuavaPass](https://guavapass.com/)**: Coaching client of [ShakaCode](http://www.shakacode.com) and React on Rails fan!
- **[GuavaPass](https://guavapass.com/)**: Coaching client of [ShakaCode](https://www.shakacode.com) and React on Rails fan!
- **[Pivotal Tracker](http://www.pivotaltracker.com/)**: The first (and most-loved) agile project management tool built on Rails. React on Rails has greatly simplified integration and workflow for our React components in Rails!
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line still uses an http:// link for Pivotal Tracker, while the rest of the updated entries in this file now use https://. Since the goal of this PR is to migrate HTTP links to HTTPS, this one appears to have been overlooked.

Suggested change
- **[Pivotal Tracker](http://www.pivotaltracker.com/)**: The first (and most-loved) agile project management tool built on Rails. React on Rails has greatly simplified integration and workflow for our React components in Rails!
- **[Pivotal Tracker](https://www.pivotaltracker.com/)**: The first (and most-loved) agile project management tool built on Rails. React on Rails has greatly simplified integration and workflow for our React components in Rails!

@justin808 justin808 merged commit b882c1a into shakacode:master Mar 5, 2026
2 checks passed
@justin808
Copy link
Copy Markdown
Member

thanks @GeObts

justin808 added a commit that referenced this pull request Mar 6, 2026
* origin/master:
  Fix private_output_path not configured on fresh Shakapacker installs (#2289)
  Bump the npm-security group across 1 directory with 3 updates (#2387)
  docs: use https links (#2518)
  Consolidate changelog to keep only rc6 prerelease (#2533)
  Fix CSS module class name divergence with rspack SSR (#2489)
  Bump version to 16.4.0.rc.6

# Conflicts:
#	CHANGELOG.md
justin808 added a commit that referenced this pull request Mar 6, 2026
* origin/master:
  Fix private_output_path not configured on fresh Shakapacker installs (#2289)
  Bump the npm-security group across 1 directory with 3 updates (#2387)
  docs: use https links (#2518)
  Consolidate changelog to keep only rc6 prerelease (#2533)
  Fix CSS module class name divergence with rspack SSR (#2489)
  Bump version to 16.4.0.rc.6
  Add BugBot license scanning config (#2515)
  Fix buildVM promise cleanup ordering (#2483) (#2484)

# Conflicts:
#	CHANGELOG.md
justin808 added a commit that referenced this pull request Mar 6, 2026
* origin/master:
  Fix private_output_path not configured on fresh Shakapacker installs (#2289)
  Bump the npm-security group across 1 directory with 3 updates (#2387)
  docs: use https links (#2518)
  Consolidate changelog to keep only rc6 prerelease (#2533)
  Fix CSS module class name divergence with rspack SSR (#2489)
  Bump version to 16.4.0.rc.6
  Add BugBot license scanning config (#2515)
  Fix buildVM promise cleanup ordering (#2483) (#2484)

# Conflicts:
#	CHANGELOG.md
AbanoubGhadban pushed a commit that referenced this pull request Mar 7, 2026
- Switch HTTP links to HTTPS in README.md and PROJECTS.md
- Each link verified to resolve over HTTPS
- Updated links: shakacode.com, madroneco.com, railsonmaui.com,
foxford.ru

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Documentation**
* Updated all external project reference links to use secure HTTPS
protocol
* Added two new organizations (LocalWise and Ellevest) to the Commercial
Products Live section
* Revised project descriptions and formatting across multiple entries to
improve consistency
  * Updated the career page link to HTTPS

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
justin808 added a commit that referenced this pull request Mar 8, 2026
…upport

* origin/master: (38 commits)
  fix: use env-var-driven ports in Procfile templates to support multiple worktrees (#2539)
  Fix prettier formatting in auto-bundling doc
  Docs: Clarify .client/.server suffixes vs use client RSC directive (#2406)
  Warn against using .server/.client variants with RSC features
  Docs: move internal-only docs out of published docs trees (#2414)
  Fix crash when HTTPX::ErrorResponse is returned in get_form_body_for_file (#2532)
  Skip flaky external URLs in lychee checks (#2547)
  Update mise docs: prefer shell-level shims over conductor-exec (#2537)
  Document compression middleware compatibility with streaming SSR (#2544)
  Fix duplicate node-renderer message reporting in render failures (#2531)
  Fix private_output_path not configured on fresh Shakapacker installs (#2289)
  Bump the npm-security group across 1 directory with 3 updates (#2387)
  docs: use https links (#2518)
  Consolidate changelog to keep only rc6 prerelease (#2533)
  Fix CSS module class name divergence with rspack SSR (#2489)
  Bump version to 16.4.0.rc.6
  Add BugBot license scanning config (#2515)
  Fix buildVM promise cleanup ordering (#2483) (#2484)
  Fix streaming SSR hangs and silent error absorption in RSC payload injection (#2407)
  Ensure lefthook uses mise tools in non-interactive shells (#2512)
  ...

# Conflicts:
#	CHANGELOG.md
justin808 added a commit that referenced this pull request Mar 9, 2026
## Summary

- Add changelog entries for 6 user-visible PRs merged since v16.4.0.rc.6
that were missing from `[Unreleased]`
- Update existing #2561 entry to include #2568 contributor credit

### New entries added

| Section | PR | Description |
|---|---|---|
| Added | #2539 | Environment-variable-driven ports in Procfile
templates |
| Fixed | #2417 | Rspack generator config path fix |
| Fixed | #2419 | Precompile hook load-based execution fix |
| Fixed | #2577 | `create-react-on-rails-app` validation improvements |
| Pro Fixed | #2416 | StreamResponse status fallback fix |
| Pro Fixed | #2566 | Empty-string license plan mismatch fix |

### Skipped PRs (not user-visible)

Docs (#2406, #2414, #2479, #2494, #2518, #2537, #2544), CI/internal
(#2533, #2547, #2555, #2557, #2558, #2564), dependabot (#2387, #2541),
dev dependencies (#2559, #2569, #2573).

## Test plan

- [ ] Verify changelog formatting matches existing entries
- [ ] Verify all user-visible PRs since v16.4.0.rc.6 are covered

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Documentation-only changelog updates with no runtime or build behavior
changes.
> 
> **Overview**
> Updates `CHANGELOG.md`’s **[Unreleased]** section to include
previously missing user-facing entries: Procfile templates now support
env-driven ports, several generator/`bin/dev` precompile-hook and
rspack-path fixes are documented, and `create-react-on-rails-app`
validation improvements are noted.
> 
> Also adds two Pro fix entries (StreamResponse status fallback and
license plan empty-string validation) and updates the existing `bin/dev`
precompile-hook entry to credit an additional PR/contributor.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
e75d2b5. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants