Skip to content

Update to use npm vite - #385

Open
tksheppard wants to merge 40 commits into
nvisionative:developfrom
tksheppard:update-to-use-npm-vite
Open

tksheppard wants to merge 40 commits into
nvisionative:developfrom
tksheppard:update-to-use-npm-vite

Conversation

@tksheppard

@tksheppard tksheppard commented Feb 19, 2026 •

Copy link
Copy Markdown
Collaborator

Related to Issue

Fixes #356

Description

This PR aims to update the build system and task automation to leverage Vite. This should help compilation speeds greatly. And bring us up-to-date on technology. New docs made, old docs are kept as reference in case we want to reuse anything (can eventually be removed). Updated to use typescript.

How Has This Been Tested?

Needs testing again

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@tksheppard tksheppard added this to the 4.0.0 milestone Feb 19, 2026
@tksheppard tksheppard self-assigned this Feb 19, 2026
@tksheppard tksheppard added the enhancement New feature or request label Feb 19, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard why was this file deleted? This is used for the GitHub issue template (which should still be valid).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in bac22c0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard why was this file deleted? This is used for the GitHub issue template (which should still be valid).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in bac22c0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard why was this file deleted? This is used the GitHub Actions CI build.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in bac22c0

Comment thread .github/FUNDING.yml

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard why was this file deleted? This is used for GitHub Sponsors.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in bac22c0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard why was this file deleted? This is used for the GitHub pull request template (which should still be valid).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in bac22c0

Comment thread LICENSE.md

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard is there a reason why this file name changed? I am not sure if GitHub supports it with this naming. GitHub has a feature to auto-detect the LICENSE file and display the correct license type in other areas of the GitHub repo UI.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@david-poindexter I was going for consistency with all markdown files--and GitHub supports it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard it is good to know GitHub supports it, but I still prefer this file to be treated differently than other markdown files. It is "special" because it is the license. Thanks.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated in da05835

Comment thread project-details.json Resolved
Comment thread README.md Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard why was this file deleted? This GitHub repo needs a README.md.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@david-poindexter The file wasn't deleted, it was moved to the .github folder, which loads correctly in github. This keeps the root folder cleaner.

Comment thread vite.config.js Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard as a first step, please convert this to TypeScript. I'll review it more granularly after that.

Comment thread serve.config.js Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread scripts/build.js Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread scripts/clean.ts

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread scripts/helpers.js Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread scripts/package.js Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread scripts/postbuild.js Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread scripts/serve.js Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread scripts/utils.js Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread scripts/watch.ts

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

Comment thread src/js/custom.js

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tksheppard please convert this to TypeScript.

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

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace gulp with vite

3 participants