fix: post Codecov results as GitHub Checks, not only commit statuses - #6044

Closed
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks
Closed

fix: post Codecov results as GitHub Checks, not only commit statuses#6044
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks

Conversation

@jam-jee

Copy link
Copy Markdown
Collaborator

Issue

Codecov coverage results are no longer visible on the PR Checks tab. Since early July 2026 Codecov posts only legacy commit statuses (codecov/project, codecov/patch) for this repo — these render under the PR Conversation merge box ("Show all checks") but not on /checks.

Verified via the GitHub API across PRs:

PRCreatedCodecov check-runsCodecov commit statuses
#5969Jun 262 (on /checks)0
#6004Jul 0806
#6041Jul 1705

The switch happened between late June and early July and affects every PR — including ones carrying no codecov.yml at all.

Root cause

GitHub Checks are Codecov's default behavior (no yaml required), so the fact that config-less PRs also lost them indicates the regression is at the GitHub App integration level — the Codecov GitHub App losing its "Checks: read & write" permission on the repo (e.g. an app reinstall or a GitHub-required permission update that was never re-accepted). When Codecov cannot write Checks, it silently falls back to commit statuses.

Fix (two parts)

1. This PR (documents intent + re-enables annotations): adds

github_checks:
annotations: true

Checks are already the default, so this is explicit-intent + re-enables line annotations once the app permission is restored. The patch status (required for checks to appear) is already configured.

2. Required admin action (not in this PR): an org/repo admin must re-authorize the Codecov GitHub App so it regains Checks permission:

  • GitHub → aws org → Settings → GitHub Apps → Codecov → Configure — accept any pending permission request / ensure Checks: Read & write is granted for sagemaker-python-sdk, or
  • reconnect the integration from the Codecov dashboard (app.codecov.io → repo → Settings).

This yaml change alone will not move results back to the Checks tab without step 2.

Testing

  • python -c "import yaml; yaml.safe_load(open('codecov.yml'))" → parses.
  • Codecov config validator (curl --data-binary @codecov.yml https://codecov.io/validate) → Valid!.

Since early July 2026 Codecov has been posting only legacy commit statuses
(`codecov/project`, `codecov/patch`) for this repo. These appear under the
PR "Conversation" merge box but NOT on the PR "Checks" tab, so reviewers
looking at /checks no longer see coverage results. Earlier PRs (e.g. aws#5969,
Jun 26) correctly posted GitHub check-runs visible on /checks.
GitHub Checks are Codecov's default, so this regression is at the GitHub App
integration level (the Codecov app losing "Checks: read & write" permission),
not a codecov.yml setting -- it affects even PRs that carry no codecov.yml.
Add an explicit `github_checks: annotations: true` to document the intended
behavior and re-enable check annotations once the app permission is restored.
NOTE: this yaml alone does not fully fix visibility. An org/repo admin must
re-authorize the Codecov GitHub App so it regains Checks permission
(GitHub > aws org > Settings > GitHub Apps > Codecov > Configure, or reconnect
the integration from the Codecov dashboard).
Sign up for freeto 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.

1 participant

@jam-jee
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

fix: post Codecov results as GitHub Checks, not only commit statuses - #6044

Closed
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks
Closed

fix: post Codecov results as GitHub Checks, not only commit statuses#6044
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks

Conversation

@jam-jee

Copy link
Copy Markdown
Collaborator

Issue

Codecov coverage results are no longer visible on the PR Checks tab. Since early July 2026 Codecov posts only legacy commit statuses (codecov/project, codecov/patch) for this repo — these render under the PR Conversation merge box ("Show all checks") but not on /checks.

Verified via the GitHub API across PRs:

PRCreatedCodecov check-runsCodecov commit statuses
#5969Jun 262 (on /checks)0
#6004Jul 0806
#6041Jul 1705

The switch happened between late June and early July and affects every PR — including ones carrying no codecov.yml at all.

Root cause

GitHub Checks are Codecov's default behavior (no yaml required), so the fact that config-less PRs also lost them indicates the regression is at the GitHub App integration level — the Codecov GitHub App losing its "Checks: read & write" permission on the repo (e.g. an app reinstall or a GitHub-required permission update that was never re-accepted). When Codecov cannot write Checks, it silently falls back to commit statuses.

Fix (two parts)

1. This PR (documents intent + re-enables annotations): adds

github_checks:
annotations: true

Checks are already the default, so this is explicit-intent + re-enables line annotations once the app permission is restored. The patch status (required for checks to appear) is already configured.

2. Required admin action (not in this PR): an org/repo admin must re-authorize the Codecov GitHub App so it regains Checks permission:

  • GitHub → aws org → Settings → GitHub Apps → Codecov → Configure — accept any pending permission request / ensure Checks: Read & write is granted for sagemaker-python-sdk, or
  • reconnect the integration from the Codecov dashboard (app.codecov.io → repo → Settings).

This yaml change alone will not move results back to the Checks tab without step 2.

Testing

  • python -c "import yaml; yaml.safe_load(open('codecov.yml'))" → parses.
  • Codecov config validator (curl --data-binary @codecov.yml https://codecov.io/validate) → Valid!.

Since early July 2026 Codecov has been posting only legacy commit statuses
(`codecov/project`, `codecov/patch`) for this repo. These appear under the
PR "Conversation" merge box but NOT on the PR "Checks" tab, so reviewers
looking at /checks no longer see coverage results. Earlier PRs (e.g. aws#5969,
Jun 26) correctly posted GitHub check-runs visible on /checks.
GitHub Checks are Codecov's default, so this regression is at the GitHub App
integration level (the Codecov app losing "Checks: read & write" permission),
not a codecov.yml setting -- it affects even PRs that carry no codecov.yml.
Add an explicit `github_checks: annotations: true` to document the intended
behavior and re-enable check annotations once the app permission is restored.
NOTE: this yaml alone does not fully fix visibility. An org/repo admin must
re-authorize the Codecov GitHub App so it regains Checks permission
(GitHub > aws org > Settings > GitHub Apps > Codecov > Configure, or reconnect
the integration from the Codecov dashboard).
Sign up for freeto 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.

1 participant

@jam-jee
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix: post Codecov results as GitHub Checks, not only commit statuses - #6044

Closed
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks
Closed

fix: post Codecov results as GitHub Checks, not only commit statuses#6044
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks

Conversation

@jam-jee

Copy link
Copy Markdown
Collaborator

Issue

Codecov coverage results are no longer visible on the PR Checks tab. Since early July 2026 Codecov posts only legacy commit statuses (codecov/project, codecov/patch) for this repo — these render under the PR Conversation merge box ("Show all checks") but not on /checks.

Verified via the GitHub API across PRs:

PRCreatedCodecov check-runsCodecov commit statuses
#5969Jun 262 (on /checks)0
#6004Jul 0806
#6041Jul 1705

The switch happened between late June and early July and affects every PR — including ones carrying no codecov.yml at all.

Root cause

GitHub Checks are Codecov's default behavior (no yaml required), so the fact that config-less PRs also lost them indicates the regression is at the GitHub App integration level — the Codecov GitHub App losing its "Checks: read & write" permission on the repo (e.g. an app reinstall or a GitHub-required permission update that was never re-accepted). When Codecov cannot write Checks, it silently falls back to commit statuses.

Fix (two parts)

1. This PR (documents intent + re-enables annotations): adds

github_checks:
annotations: true

Checks are already the default, so this is explicit-intent + re-enables line annotations once the app permission is restored. The patch status (required for checks to appear) is already configured.

2. Required admin action (not in this PR): an org/repo admin must re-authorize the Codecov GitHub App so it regains Checks permission:

  • GitHub → aws org → Settings → GitHub Apps → Codecov → Configure — accept any pending permission request / ensure Checks: Read & write is granted for sagemaker-python-sdk, or
  • reconnect the integration from the Codecov dashboard (app.codecov.io → repo → Settings).

This yaml change alone will not move results back to the Checks tab without step 2.

Testing

  • python -c "import yaml; yaml.safe_load(open('codecov.yml'))" → parses.
  • Codecov config validator (curl --data-binary @codecov.yml https://codecov.io/validate) → Valid!.

Since early July 2026 Codecov has been posting only legacy commit statuses
(`codecov/project`, `codecov/patch`) for this repo. These appear under the
PR "Conversation" merge box but NOT on the PR "Checks" tab, so reviewers
looking at /checks no longer see coverage results. Earlier PRs (e.g. aws#5969,
Jun 26) correctly posted GitHub check-runs visible on /checks.
GitHub Checks are Codecov's default, so this regression is at the GitHub App
integration level (the Codecov app losing "Checks: read & write" permission),
not a codecov.yml setting -- it affects even PRs that carry no codecov.yml.
Add an explicit `github_checks: annotations: true` to document the intended
behavior and re-enable check annotations once the app permission is restored.
NOTE: this yaml alone does not fully fix visibility. An org/repo admin must
re-authorize the Codecov GitHub App so it regains Checks permission
(GitHub > aws org > Settings > GitHub Apps > Codecov > Configure, or reconnect
the integration from the Codecov dashboard).
Sign up for freeto 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.

1 participant

@jam-jee
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix: post Codecov results as GitHub Checks, not only commit statuses - #6044

Closed
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks
Closed

fix: post Codecov results as GitHub Checks, not only commit statuses#6044
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks

Conversation

@jam-jee

Copy link
Copy Markdown
Collaborator

Issue

Codecov coverage results are no longer visible on the PR Checks tab. Since early July 2026 Codecov posts only legacy commit statuses (codecov/project, codecov/patch) for this repo — these render under the PR Conversation merge box ("Show all checks") but not on /checks.

Verified via the GitHub API across PRs:

PRCreatedCodecov check-runsCodecov commit statuses
#5969Jun 262 (on /checks)0
#6004Jul 0806
#6041Jul 1705

The switch happened between late June and early July and affects every PR — including ones carrying no codecov.yml at all.

Root cause

GitHub Checks are Codecov's default behavior (no yaml required), so the fact that config-less PRs also lost them indicates the regression is at the GitHub App integration level — the Codecov GitHub App losing its "Checks: read & write" permission on the repo (e.g. an app reinstall or a GitHub-required permission update that was never re-accepted). When Codecov cannot write Checks, it silently falls back to commit statuses.

Fix (two parts)

1. This PR (documents intent + re-enables annotations): adds

github_checks:
annotations: true

Checks are already the default, so this is explicit-intent + re-enables line annotations once the app permission is restored. The patch status (required for checks to appear) is already configured.

2. Required admin action (not in this PR): an org/repo admin must re-authorize the Codecov GitHub App so it regains Checks permission:

  • GitHub → aws org → Settings → GitHub Apps → Codecov → Configure — accept any pending permission request / ensure Checks: Read & write is granted for sagemaker-python-sdk, or
  • reconnect the integration from the Codecov dashboard (app.codecov.io → repo → Settings).

This yaml change alone will not move results back to the Checks tab without step 2.

Testing

  • python -c "import yaml; yaml.safe_load(open('codecov.yml'))" → parses.
  • Codecov config validator (curl --data-binary @codecov.yml https://codecov.io/validate) → Valid!.

Since early July 2026 Codecov has been posting only legacy commit statuses
(`codecov/project`, `codecov/patch`) for this repo. These appear under the
PR "Conversation" merge box but NOT on the PR "Checks" tab, so reviewers
looking at /checks no longer see coverage results. Earlier PRs (e.g. aws#5969,
Jun 26) correctly posted GitHub check-runs visible on /checks.
GitHub Checks are Codecov's default, so this regression is at the GitHub App
integration level (the Codecov app losing "Checks: read & write" permission),
not a codecov.yml setting -- it affects even PRs that carry no codecov.yml.
Add an explicit `github_checks: annotations: true` to document the intended
behavior and re-enable check annotations once the app permission is restored.
NOTE: this yaml alone does not fully fix visibility. An org/repo admin must
re-authorize the Codecov GitHub App so it regains Checks permission
(GitHub > aws org > Settings > GitHub Apps > Codecov > Configure, or reconnect
the integration from the Codecov dashboard).
Sign up for freeto 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.

1 participant

@jam-jee
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

fix: post Codecov results as GitHub Checks, not only commit statuses - #6044

Closed
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks
Closed

fix: post Codecov results as GitHub Checks, not only commit statuses#6044
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks

Conversation

@jam-jee

Copy link
Copy Markdown
Collaborator

Issue

Codecov coverage results are no longer visible on the PR Checks tab. Since early July 2026 Codecov posts only legacy commit statuses (codecov/project, codecov/patch) for this repo — these render under the PR Conversation merge box ("Show all checks") but not on /checks.

Verified via the GitHub API across PRs:

PRCreatedCodecov check-runsCodecov commit statuses
#5969Jun 262 (on /checks)0
#6004Jul 0806
#6041Jul 1705

The switch happened between late June and early July and affects every PR — including ones carrying no codecov.yml at all.

Root cause

GitHub Checks are Codecov's default behavior (no yaml required), so the fact that config-less PRs also lost them indicates the regression is at the GitHub App integration level — the Codecov GitHub App losing its "Checks: read & write" permission on the repo (e.g. an app reinstall or a GitHub-required permission update that was never re-accepted). When Codecov cannot write Checks, it silently falls back to commit statuses.

Fix (two parts)

1. This PR (documents intent + re-enables annotations): adds

github_checks:
annotations: true

Checks are already the default, so this is explicit-intent + re-enables line annotations once the app permission is restored. The patch status (required for checks to appear) is already configured.

2. Required admin action (not in this PR): an org/repo admin must re-authorize the Codecov GitHub App so it regains Checks permission:

  • GitHub → aws org → Settings → GitHub Apps → Codecov → Configure — accept any pending permission request / ensure Checks: Read & write is granted for sagemaker-python-sdk, or
  • reconnect the integration from the Codecov dashboard (app.codecov.io → repo → Settings).

This yaml change alone will not move results back to the Checks tab without step 2.

Testing

  • python -c "import yaml; yaml.safe_load(open('codecov.yml'))" → parses.
  • Codecov config validator (curl --data-binary @codecov.yml https://codecov.io/validate) → Valid!.

Since early July 2026 Codecov has been posting only legacy commit statuses
(`codecov/project`, `codecov/patch`) for this repo. These appear under the
PR "Conversation" merge box but NOT on the PR "Checks" tab, so reviewers
looking at /checks no longer see coverage results. Earlier PRs (e.g. aws#5969,
Jun 26) correctly posted GitHub check-runs visible on /checks.
GitHub Checks are Codecov's default, so this regression is at the GitHub App
integration level (the Codecov app losing "Checks: read & write" permission),
not a codecov.yml setting -- it affects even PRs that carry no codecov.yml.
Add an explicit `github_checks: annotations: true` to document the intended
behavior and re-enable check annotations once the app permission is restored.
NOTE: this yaml alone does not fully fix visibility. An org/repo admin must
re-authorize the Codecov GitHub App so it regains Checks permission
(GitHub > aws org > Settings > GitHub Apps > Codecov > Configure, or reconnect
the integration from the Codecov dashboard).
Sign up for freeto 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.

1 participant

@jam-jee
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix: post Codecov results as GitHub Checks, not only commit statuses - #6044

Closed
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks
Closed

fix: post Codecov results as GitHub Checks, not only commit statuses#6044
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks

Conversation

@jam-jee

Copy link
Copy Markdown
Collaborator

Issue

Codecov coverage results are no longer visible on the PR Checks tab. Since early July 2026 Codecov posts only legacy commit statuses (codecov/project, codecov/patch) for this repo — these render under the PR Conversation merge box ("Show all checks") but not on /checks.

Verified via the GitHub API across PRs:

PRCreatedCodecov check-runsCodecov commit statuses
#5969Jun 262 (on /checks)0
#6004Jul 0806
#6041Jul 1705

The switch happened between late June and early July and affects every PR — including ones carrying no codecov.yml at all.

Root cause

GitHub Checks are Codecov's default behavior (no yaml required), so the fact that config-less PRs also lost them indicates the regression is at the GitHub App integration level — the Codecov GitHub App losing its "Checks: read & write" permission on the repo (e.g. an app reinstall or a GitHub-required permission update that was never re-accepted). When Codecov cannot write Checks, it silently falls back to commit statuses.

Fix (two parts)

1. This PR (documents intent + re-enables annotations): adds

github_checks:
annotations: true

Checks are already the default, so this is explicit-intent + re-enables line annotations once the app permission is restored. The patch status (required for checks to appear) is already configured.

2. Required admin action (not in this PR): an org/repo admin must re-authorize the Codecov GitHub App so it regains Checks permission:

  • GitHub → aws org → Settings → GitHub Apps → Codecov → Configure — accept any pending permission request / ensure Checks: Read & write is granted for sagemaker-python-sdk, or
  • reconnect the integration from the Codecov dashboard (app.codecov.io → repo → Settings).

This yaml change alone will not move results back to the Checks tab without step 2.

Testing

  • python -c "import yaml; yaml.safe_load(open('codecov.yml'))" → parses.
  • Codecov config validator (curl --data-binary @codecov.yml https://codecov.io/validate) → Valid!.

Since early July 2026 Codecov has been posting only legacy commit statuses
(`codecov/project`, `codecov/patch`) for this repo. These appear under the
PR "Conversation" merge box but NOT on the PR "Checks" tab, so reviewers
looking at /checks no longer see coverage results. Earlier PRs (e.g. aws#5969,
Jun 26) correctly posted GitHub check-runs visible on /checks.
GitHub Checks are Codecov's default, so this regression is at the GitHub App
integration level (the Codecov app losing "Checks: read & write" permission),
not a codecov.yml setting -- it affects even PRs that carry no codecov.yml.
Add an explicit `github_checks: annotations: true` to document the intended
behavior and re-enable check annotations once the app permission is restored.
NOTE: this yaml alone does not fully fix visibility. An org/repo admin must
re-authorize the Codecov GitHub App so it regains Checks permission
(GitHub > aws org > Settings > GitHub Apps > Codecov > Configure, or reconnect
the integration from the Codecov dashboard).
Sign up for freeto 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.

1 participant

@jam-jee
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix: post Codecov results as GitHub Checks, not only commit statuses - #6044

Closed
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks
Closed

fix: post Codecov results as GitHub Checks, not only commit statuses#6044
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks

Conversation

@jam-jee

Copy link
Copy Markdown
Collaborator

Issue

Codecov coverage results are no longer visible on the PR Checks tab. Since early July 2026 Codecov posts only legacy commit statuses (codecov/project, codecov/patch) for this repo — these render under the PR Conversation merge box ("Show all checks") but not on /checks.

Verified via the GitHub API across PRs:

PRCreatedCodecov check-runsCodecov commit statuses
#5969Jun 262 (on /checks)0
#6004Jul 0806
#6041Jul 1705

The switch happened between late June and early July and affects every PR — including ones carrying no codecov.yml at all.

Root cause

GitHub Checks are Codecov's default behavior (no yaml required), so the fact that config-less PRs also lost them indicates the regression is at the GitHub App integration level — the Codecov GitHub App losing its "Checks: read & write" permission on the repo (e.g. an app reinstall or a GitHub-required permission update that was never re-accepted). When Codecov cannot write Checks, it silently falls back to commit statuses.

Fix (two parts)

1. This PR (documents intent + re-enables annotations): adds

github_checks:
annotations: true

Checks are already the default, so this is explicit-intent + re-enables line annotations once the app permission is restored. The patch status (required for checks to appear) is already configured.

2. Required admin action (not in this PR): an org/repo admin must re-authorize the Codecov GitHub App so it regains Checks permission:

  • GitHub → aws org → Settings → GitHub Apps → Codecov → Configure — accept any pending permission request / ensure Checks: Read & write is granted for sagemaker-python-sdk, or
  • reconnect the integration from the Codecov dashboard (app.codecov.io → repo → Settings).

This yaml change alone will not move results back to the Checks tab without step 2.

Testing

  • python -c "import yaml; yaml.safe_load(open('codecov.yml'))" → parses.
  • Codecov config validator (curl --data-binary @codecov.yml https://codecov.io/validate) → Valid!.

Since early July 2026 Codecov has been posting only legacy commit statuses
(`codecov/project`, `codecov/patch`) for this repo. These appear under the
PR "Conversation" merge box but NOT on the PR "Checks" tab, so reviewers
looking at /checks no longer see coverage results. Earlier PRs (e.g. aws#5969,
Jun 26) correctly posted GitHub check-runs visible on /checks.
GitHub Checks are Codecov's default, so this regression is at the GitHub App
integration level (the Codecov app losing "Checks: read & write" permission),
not a codecov.yml setting -- it affects even PRs that carry no codecov.yml.
Add an explicit `github_checks: annotations: true` to document the intended
behavior and re-enable check annotations once the app permission is restored.
NOTE: this yaml alone does not fully fix visibility. An org/repo admin must
re-authorize the Codecov GitHub App so it regains Checks permission
(GitHub > aws org > Settings > GitHub Apps > Codecov > Configure, or reconnect
the integration from the Codecov dashboard).
Sign up for freeto 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.

1 participant

@jam-jee
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

fix: post Codecov results as GitHub Checks, not only commit statuses - #6044

Closed
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks
Closed

fix: post Codecov results as GitHub Checks, not only commit statuses#6044
jam-jee wants to merge 1 commit into
aws:masterfrom
jam-jee:fix/codecov-github-checks

Conversation

@jam-jee

Copy link
Copy Markdown
Collaborator

Issue

Codecov coverage results are no longer visible on the PR Checks tab. Since early July 2026 Codecov posts only legacy commit statuses (codecov/project, codecov/patch) for this repo — these render under the PR Conversation merge box ("Show all checks") but not on /checks.

Verified via the GitHub API across PRs:

PRCreatedCodecov check-runsCodecov commit statuses
#5969Jun 262 (on /checks)0
#6004Jul 0806
#6041Jul 1705

The switch happened between late June and early July and affects every PR — including ones carrying no codecov.yml at all.

Root cause

GitHub Checks are Codecov's default behavior (no yaml required), so the fact that config-less PRs also lost them indicates the regression is at the GitHub App integration level — the Codecov GitHub App losing its "Checks: read & write" permission on the repo (e.g. an app reinstall or a GitHub-required permission update that was never re-accepted). When Codecov cannot write Checks, it silently falls back to commit statuses.

Fix (two parts)

1. This PR (documents intent + re-enables annotations): adds

github_checks:
annotations: true

Checks are already the default, so this is explicit-intent + re-enables line annotations once the app permission is restored. The patch status (required for checks to appear) is already configured.

2. Required admin action (not in this PR): an org/repo admin must re-authorize the Codecov GitHub App so it regains Checks permission:

  • GitHub → aws org → Settings → GitHub Apps → Codecov → Configure — accept any pending permission request / ensure Checks: Read & write is granted for sagemaker-python-sdk, or
  • reconnect the integration from the Codecov dashboard (app.codecov.io → repo → Settings).

This yaml change alone will not move results back to the Checks tab without step 2.

Testing

  • python -c "import yaml; yaml.safe_load(open('codecov.yml'))" → parses.
  • Codecov config validator (curl --data-binary @codecov.yml https://codecov.io/validate) → Valid!.

Since early July 2026 Codecov has been posting only legacy commit statuses
(`codecov/project`, `codecov/patch`) for this repo. These appear under the
PR "Conversation" merge box but NOT on the PR "Checks" tab, so reviewers
looking at /checks no longer see coverage results. Earlier PRs (e.g. aws#5969,
Jun 26) correctly posted GitHub check-runs visible on /checks.
GitHub Checks are Codecov's default, so this regression is at the GitHub App
integration level (the Codecov app losing "Checks: read & write" permission),
not a codecov.yml setting -- it affects even PRs that carry no codecov.yml.
Add an explicit `github_checks: annotations: true` to document the intended
behavior and re-enable check annotations once the app permission is restored.
NOTE: this yaml alone does not fully fix visibility. An org/repo admin must
re-authorize the Codecov GitHub App so it regains Checks permission
(GitHub > aws org > Settings > GitHub Apps > Codecov > Configure, or reconnect
the integration from the Codecov dashboard).
Sign up for freeto 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.

1 participant

@jam-jee