Skip to content

Fix: remove all barrel imports and unused deps - #89

Open
santipalenque wants to merge 1 commit into
masterfrom
fix/remove-barrel-imports
Open

Fix: remove all barrel imports and unused deps#89
santipalenque wants to merge 1 commit into
masterfrom
fix/remove-barrel-imports

Conversation

@santipalenque

@santipalenque santipalenque commented Jul 17, 2026

Copy link
Copy Markdown

https://app.clickup.com/t/9014802374/86baz0zt6

Summary by CodeRabbit

  • Maintenance
    • Updated the shared UI foundation to a newer version.
    • Improved component integration across forms and pages by using supported component imports.
  • Bug Fixes
    • Preserved existing loading, form, selection, presentation, and clock functionality while improving compatibility with the updated UI foundation.

@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 49ca76c1-cf03-46a3-8fab-49203f1cd3cc

📥 Commits

Reviewing files that changed from the base of the PR and between 7b4393f and 1d559ab.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (17)
  • package.json
  • src/app.js
  • src/components/affiliationstable/AffiliationsTable.js
  • src/components/clock.js
  • src/components/inputs/questions-input.js
  • src/components/presentation-review-form.js
  • src/components/presentation-speakers-form.js
  • src/components/presentation-summary-form.js
  • src/components/presentation-uploads-form.js
  • src/components/speaker-form.js
  • src/pages/landing-page.js
  • src/pages/plan-selection-page.js
  • src/pages/preview-presentation-page.js
  • src/pages/selection-process-page.js
  • src/pages/summit-selection-page.js
  • src/pages/thankyou-presentation-page.js
  • src/pages/tracks-guide-page.js

📝 Walkthrough

Walkthrough

The pull request upgrades openstack-uicore-foundation to 5.0.43 and replaces barrel imports with direct component-module imports across application, component, and page files. Runtime logic, rendering, and exported APIs remain unchanged.

Changes

UI Core Foundation import migration

Layer / File(s) Summary
Dependency upgrade
package.json
Updates openstack-uicore-foundation from 4.2.28-beta.6 to 5.0.43.
Application and component imports
src/app.js, src/components/*
Replaces aggregated UI component imports with direct module paths without changing component behavior.
Page imports
src/pages/*
Updates Exclusive, Dropdown, and RawHTML imports to their specific module paths while preserving page behavior.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: smarcet

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: replacing barrel imports with direct module imports across the project.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/remove-barrel-imports

Comment @coderabbitai help to get the list of available commands.

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