Skip to content

CI: fix dead scrutinizer-ci.com/ocular.phar URL - #115

Merged
aik099 merged 1 commit into
masterfrom
ci/fix-scrutinizer-ocular-url
Jul 23, 2026
Merged

CI: fix dead scrutinizer-ci.com/ocular.phar URL#115
aik099 merged 1 commit into
masterfrom
ci/fix-scrutinizer-ocular-url

Conversation

@aik099

@aik099 aik099 commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Problem

Every CI job currently fails, on every branch/PR, regardless of the actual code change. The "Upload Coverage to Scrutinizer CI (PHP < 8.0)" step runs:

wget https://scrutinizer-ci.com/ocular.phar

which now returns 403 Forbidden — this endpoint appears to be dead. wget exits non-zero and fails the whole job, even though vendor/bin/phpunit itself passes cleanly right before it (visible in the logs of every currently-failing job, e.g. PR #114).

Only the PHP < 8.0 step is affected — the PHP >= 8.0 step installs scrutinizer/ocular via Composer instead and never hits that URL.

Fix

Point wget at the GitHub Releases copy of ocular.phar instead:

-  wget https://scrutinizer-ci.com/ocular.phar
+  wget https://github.com/scrutinizer-ci/ocular/releases/download/1.6.0/ocular.phar

Verified the URL resolves and serves the file (200 OK). Same fix already applied in a sibling repo, console-helpers/svn-buddy (commit 7a34548, "Use "ocular.phar" from GitHub CDN").

Closes #112

The "Upload Coverage to Scrutinizer CI (PHP < 8.0)" step downloads
ocular.phar via wget from scrutinizer-ci.com, which now returns
403 Forbidden, failing every CI job on every PHP version even though
the actual test run passes.

Point wget at the GitHub Releases copy of ocular.phar instead, same
fix already applied in console-helpers/svn-buddy.
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.73%. Comparing base (8061167) to head (cdd8696).

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #115   +/-   ##
=========================================
  Coverage     97.73%   97.73%           
  Complexity      498      498           
=========================================
  Files            29       29           
  Lines          1460     1460           
=========================================
  Hits           1427     1427           
  Misses           33       33           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aik099
aik099 merged commit fe2bae3 into master Jul 23, 2026
18 of 19 checks passed
@aik099
aik099 deleted the ci/fix-scrutinizer-ocular-url branch July 23, 2026 11:13
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.

Change "ocular.phar" download origin

1 participant