Skip to content

Updated gitignore and modernized vite config#19

Merged
blstmo merged 3 commits intofluxerjs:mainfrom
decker82:minor-fixes
Feb 22, 2026
Merged

Updated gitignore and modernized vite config#19
blstmo merged 3 commits intofluxerjs:mainfrom
decker82:minor-fixes

Conversation

@decker82
Copy link
Contributor

@decker82 decker82 commented Feb 20, 2026

Description

Updated gitignore and modernized vite config (no changes to the logic).

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Checklist

  • My code follows the project's style guidelines (run pnpm run lint)
  • I have run pnpm run build successfully
  • I have run pnpm run test successfully

Summary by CodeRabbit

  • Chores
    • Updated build artifact ignoring configuration to exclude additional Vite artifacts.
    • Improved build configuration code style.

@vercel
Copy link

vercel bot commented Feb 20, 2026

@decker82 is attempting to deploy a commit to the blstmo's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Feb 21, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

Two configuration updates: addition of a Vite build artifact ignore pattern to .gitignore, and refactoring of the closeBundle method syntax in the Vite configuration from legacy function syntax to modern method shorthand with const variable declaration.

Changes

Cohort / File(s) Summary
Build Configuration
.gitignore
Added ignore pattern for Vite build artifacts: *.timestamp-*.mjs
Plugin Method Refactoring
apps/docs/vite.config.js
Updated closeBundle method syntax from closeBundle: function () { var outDir... to closeBundle() { const outDir...}, modernizing function syntax and variable declaration strategy.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A timestamp file hops out of sight,
While methods don their modern light—
From var to const, the syntax gleams,
No logic changed, just cleaner dreams! ✨

🚥 Pre-merge checks | ✅ 3
✅ 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 accurately summarizes the main changes: updates to .gitignore and modernization of vite config, matching the commit messages and file modifications.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

@blstmo blstmo merged commit 420d63d into fluxerjs:main Feb 22, 2026
1 of 2 checks passed
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.

2 participants