Update formula#203
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Duplication | 0 |
🟢 Coverage ∅ diff coverage · +0.00% coverage variation
Metric Results Coverage variation ✅ +0.00% coverage variation (-0.50%) Diff coverage ✅ ∅ diff coverage (50.00%) Coverage variation details
Coverable lines Covered lines Coverage Common ancestor commit (75d97e9) 6211 1502 24.18% Head commit (36e6a51) 6211 (+0) 1502 (+0) 24.18% (+0.00%) Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch:
<coverage of head commit> - <coverage of common ancestor commit>Diff coverage details
Coverable lines Covered lines Diff coverage Pull request (#203) 0 0 ∅ (not applicable) Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified:
<covered lines added or modified>/<coverable lines added or modified> * 100%
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
Code Review
This pull request updates the version and URL for the codacy-cli-v2 Homebrew formula. Feedback highlights that the updated URL contains an incorrect .sha. segment which will likely result in a 404 error, and notes that the sha256 checksum must be updated to match the new file.
| version "1.0.0-main.275.bc32470" | ||
| url "https://raw.githubusercontent.com/codacy/codacy-cli-v2/1.0.0-main.275.bc32470/codacy-cli.sh" | ||
| version "1.0.0-main.377.75d97e9" | ||
| url "https://raw.githubusercontent.com/codacy/codacy-cli-v2/1.0.0-main.377.sha.75d97e9/codacy-cli.sh" |
There was a problem hiding this comment.
The URL contains an extra .sha. segment (1.0.0-main.377.sha.75d97e9) which does not match the version 1.0.0-main.377.75d97e9, likely resulting in a 404 Not Found error. Additionally, the sha256 checksum on line 4 must be updated to match the new file's hash to prevent Homebrew installation failures due to a checksum mismatch.
url "https://raw.githubusercontent.com/codacy/codacy-cli-v2/1.0.0-main.377.75d97e9/codacy-cli.sh"There was a problem hiding this comment.
Pull request overview
Updates the Homebrew formula for codacy-cli-v2 to point to a newer build of the codacy-cli.sh script.
Changes:
- Bumped the formula
versionto1.0.0-main.377.75d97e9 - Updated the download
urlto a new GitHub raw ref (but currently inconsistent with the version)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| version "1.0.0-main.275.bc32470" | ||
| url "https://raw.githubusercontent.com/codacy/codacy-cli-v2/1.0.0-main.275.bc32470/codacy-cli.sh" | ||
| version "1.0.0-main.377.75d97e9" | ||
| url "https://raw.githubusercontent.com/codacy/codacy-cli-v2/1.0.0-main.377.sha.75d97e9/codacy-cli.sh" |
| version "1.0.0-main.377.75d97e9" | ||
| url "https://raw.githubusercontent.com/codacy/codacy-cli-v2/1.0.0-main.377.sha.75d97e9/codacy-cli.sh" | ||
| sha256 "def852e43b05871b1a5e14dc8fb32015d89e4f4ed33fc3de8570738e6bd180f2" |
No description provided.