Skip to content

Change RadioLib to commit zip til 7.4.1+ is released#8610

Merged
caveman99 merged 1 commit into
developfrom
fix-ldro
Nov 10, 2025
Merged

Change RadioLib to commit zip til 7.4.1+ is released#8610
caveman99 merged 1 commit into
developfrom
fix-ldro

Conversation

@caveman99

Copy link
Copy Markdown
Member

fixes regression for SX127x chips per @GUVWAF

fixes regression for SX127x chips per @GUVWAF
@caveman99 caveman99 added the bugfix Pull request that fixes bugs label Nov 10, 2025
@caveman99 caveman99 requested a review from Copilot November 10, 2025 15:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR switches the RadioLib dependency from a versioned PlatformIO registry package to a specific GitHub commit archive.

  • Replaces jgromes/RadioLib@7.4.0 with a direct GitHub archive URL pointing to a specific commit (536c726)
  • Comments out the old versioned dependency instead of removing it

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread platformio.ini
@@ -115,7 +115,8 @@ lib_deps =
[radiolib_base]
lib_deps =
# renovate: datasource=custom.pio depName=RadioLib packageName=jgromes/library/RadioLib

Copilot AI Nov 10, 2025

Copy link

Choose a reason for hiding this comment

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

The renovate comment format is inconsistent with other git-based dependencies in this file. Other GitHub archive dependencies use datasource=git-refs instead of datasource=custom.pio, and they include the full package URL in packageName along with a gitBranch parameter.

For consistency with lines 64-73 and 123-124, this should be:

# renovate: datasource=git-refs depName=RadioLib packageName=https://github.com/jgromes/RadioLib gitBranch=master
https://github.com/jgromes/RadioLib/archive/536c7267362e2c1345be7054ba45e503252975ff.zip
Suggested change
# renovate: datasource=custom.pio depName=RadioLib packageName=jgromes/library/RadioLib
# renovate: datasource=git-refs depName=RadioLib packageName=https://github.com/jgromes/RadioLib gitBranch=master

Copilot uses AI. Check for mistakes.
@caveman99 caveman99 merged commit a62fed3 into develop Nov 10, 2025
82 of 83 checks passed
@caveman99 caveman99 deleted the fix-ldro branch November 10, 2025 21:31
jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
Change RadioLib to commit zip til 7.4.1+ is released
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants