Skip to content

chore: removed utils on webinar#225

Merged
hmbanan666 merged 1 commit intomainfrom
webinar
Oct 15, 2025
Merged

chore: removed utils on webinar#225
hmbanan666 merged 1 commit intomainfrom
webinar

Conversation

@hmbanan666
Copy link
Copy Markdown
Collaborator

@hmbanan666 hmbanan666 commented Oct 15, 2025

Summary by CodeRabbit

  • Refactor

    • Simplified the webinar app by removing legacy authentication and state store integrations, streamlining configuration and startup behavior.
  • Chores

    • Cleaned up dependencies to align with the refactor, removing unneeded packages.
    • Updated workspace linting preset to the latest major version for consistency with current tooling.

No new user-facing features were added; these changes focus on reducing complexity and keeping the project tooling up to date.

@hmbanan666 hmbanan666 self-assigned this Oct 15, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 15, 2025

Walkthrough

The PR removes Nuxt modules and Pinia configuration from apps/webinar/nuxt.config.ts, drops related dependencies from apps/webinar/package.json, and updates the @antfu/eslint-config catalog version in pnpm-workspace.yaml.

Changes

Cohort / File(s) Summary of Changes
Nuxt config cleanup
apps/webinar/nuxt.config.ts
Removed modules entries nuxt-auth-utils and @pinia/nuxt; deleted pinia config block (including storesDirs).
Dependencies cleanup
apps/webinar/package.json
Removed dependencies: @pinia/nuxt, pinia; removed devDependency: nuxt-auth-utils. Minor reordering only.
Workspace config bump
pnpm-workspace.yaml
Updated catalog version for @antfu/eslint-config from ^5.4.1 to ^6.0.0.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

I thump my paws, configs trimmed neat,
Pinia hops away on silent feet.
Nuxt sheds modules, light as air,
Linting fresh with updated flair.
In tidy fields, my builds now run—
Fewer carrots, same bright sun. 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title indicates the removal of utilities in the webinar app, which aligns with the deletion of the nuxt-auth-utils module but does not mention the broader removal of Pinia modules and configuration, making it only partially descriptive of the main changes.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch webinar

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a3e2568 and 0c68985.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (3)
  • apps/webinar/nuxt.config.ts (0 hunks)
  • apps/webinar/package.json (1 hunks)
  • pnpm-workspace.yaml (1 hunks)
💤 Files with no reviewable changes (1)
  • apps/webinar/nuxt.config.ts
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build
🔇 Additional comments (2)
apps/webinar/package.json (1)

20-25: No leftover Pinia references found

Scanned apps/webinar for pinia, defineStore, and createPinia; no usages remain. Safe to merge.

pnpm-workspace.yaml (1)

6-6: Node engine requirement satisfied
package.json enforces Node >=22.18.0, exceeding the ≥18 required by @antfu/eslint-config v6.0.0; ESLint is pinned to ^9.37.0. No further action needed.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​antfu/​eslint-config@​5.4.1 ⏵ 6.0.097 +110010094 -2100

View full report

@sonarqubecloud
Copy link
Copy Markdown

@hmbanan666 hmbanan666 merged commit 07200ca into main Oct 15, 2025
8 checks passed
@hmbanan666 hmbanan666 deleted the webinar branch October 15, 2025 08:16
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