Skip to content

fix: upgrade http:// to https:// in README.md#4470

Open
hanu-14 wants to merge 1 commit into
less:masterfrom
hanu-14:fix-https-readme-md
Open

fix: upgrade http:// to https:// in README.md#4470
hanu-14 wants to merge 1 commit into
less:masterfrom
hanu-14:fix-https-readme-md

Conversation

@hanu-14

@hanu-14 hanu-14 commented Jul 18, 2026

Copy link
Copy Markdown

Upgraded 6 http:// links to https:// for security in README.md.

Summary by CodeRabbit

  • Documentation
    • Updated external README links to use secure HTTPS connections.
    • Refreshed links for Less resources, development guidance, Grunt, Stack Overflow, and licensing information.

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 18, 2026
@coderabbitai

coderabbitai Bot commented Jul 18, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

README.md updates external image and hyperlink references from HTTP to HTTPS, including documentation, contribution, license, and StackOverflow links.

Changes

README link updates

Layer / File(s) Summary
Update README external URLs
README.md
Logo, documentation, contribution, license, and StackOverflow references use HTTPS URLs.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the README link protocol upgrades.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@greptile-apps

greptile-apps Bot commented Jul 18, 2026

Copy link
Copy Markdown

Greptile Summary

This PR upgrades six links in the root README to HTTPS. The changes include:

  • Secure links for the Less website, logo, and development guide.
  • Secure links for Grunt, Alexis Sellier, and Stack Overflow.
  • Unchanged hosts, paths, fragments, and link syntax.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed documentation.

Reviews (1): Last reviewed commit: "fix: upgrade http:// to https:// in READ..." | Re-trigger Greptile

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.md`:
- Line 1: Add an appropriate alt attribute to the logo img element in README.md,
preserving the existing image source, dimensions, and surrounding markup.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9d46439a-1b6b-43de-ae7e-f7a1326c0375

📥 Commits

Reviewing files that changed from the base of the PR and between 4d6e67e and c3e1781.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md
@@ -1,4 +1,4 @@
<p align="center"><img src="http://lesscss.org/public/img/less_logo.png" width="264" height="117"></p>
<p align="center"><img src="https://lesscss.org/public/img/less_logo.png" width="264" height="117"></p>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Add alternate text to the logo image.

The image has no alt attribute, so screen-reader users receive no accessible description.

Proposed fix
-<p align="center"><img src="https://lesscss.org/public/img/less_logo.png" width="264" height="117"></p>
+<p align="center"><img src="https://lesscss.org/public/img/less_logo.png" alt="Less" width="264" height="117"></p>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<p align="center"><img src="https://lesscss.org/public/img/less_logo.png" width="264" height="117"></p>
<p align="center"><img src="https://lesscss.org/public/img/less_logo.png" alt="Less" width="264" height="117"></p>
🧰 Tools
🪛 markdownlint-cli2 (0.23.0)

[warning] 1-1: Images should have alternate text (alt text)

(MD045, no-alt-text)

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` at line 1, Add an appropriate alt attribute to the logo img
element in README.md, preserving the existing image source, dimensions, and
surrounding markup.

Source: Linters/SAST tools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant