Skip to content

chore: remove static learning paths module in favor of app-defaults - #5616

Open
Zaperex wants to merge 6 commits into
redhat-developer:mainfrom
Zaperex:move-learning-paths
Open

Zaperex wants to merge 6 commits into
redhat-developer:mainfrom
Zaperex:move-learning-paths

Conversation

@Zaperex

@Zaperex Zaperex commented Oct 8, 2026

Copy link
Copy Markdown
Member

Description

Removes the static learning paths module since the app-defaults plugin includes it now

Which issue(s) does this PR fix

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Assisted-By: Codex
Signed-off-by: Frank Kong <frkong@redhat.com>

rh-pre-commit.version: 2.4.0
rh-pre-commit.check-secrets: ENABLED
@openshift-ci
openshift-ci Bot requested review from HusneShabbir and ciiay October 8, 2026 06:41
@codecov

codecov Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.80%. Comparing base (37772c4) to head (a67dc7f).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5616      +/-   ##
==========================================
- Coverage   72.40%   68.80%   -3.60%     
==========================================
  Files          68       57      -11     
  Lines         790      545     -245     
  Branches      113       83      -30     
==========================================
- Hits          572      375     -197     
+ Misses        217      169      -48     
  Partials        1        1              
Flag Coverage Δ
rhdh 68.80% <ø> (-3.60%) ⬇️
Components Coverage Δ
Backend plugins ∅ <ø> (∅)
Backend app 78.94% <ø> (ø)
Frontend app 60.40% <ø> (+2.96%) ⬆️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37772c4...a67dc7f. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

rh-pre-commit.version: 2.4.0
rh-pre-commit.check-secrets: ENABLED
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

rh-pre-commit.version: 2.4.0
rh-pre-commit.check-secrets: ENABLED
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

rh-pre-commit.version: 2.4.0
rh-pre-commit.check-secrets: ENABLED
…ve-learning-paths

rh-pre-commit.version: 2.4.0
rh-pre-commit.check-secrets: ENABLED
@sonarqubecloud

sonarqubecloud Bot commented Oct 8, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@openshift-ci

openshift-ci Bot commented Oct 8, 2026

Copy link
Copy Markdown

@Zaperex: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-ocp-helm a67dc7f link true /test e2e-ocp-helm

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

This branch has not been deployed

No deployments
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.

1 participant