-
Notifications
You must be signed in to change notification settings - Fork 9
manual build option, npm i, and remove #each keys #24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
manual build
disable cache
| on: | ||
| push: | ||
| branches: [main] | ||
| workflow_dispatch: # allows manual trigger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this? I dont want this to be run outside of a push to main
| with: | ||
| node-version-file: .nvmrc | ||
| cache: npm # cache node_modules for faster builds | ||
| # cache: npm # cache node_modules for faster builds |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this comment
|
|
||
| - name: Install dependencies | ||
| run: npm ci # ci (not install) to install exact versions from package-lock.json | ||
| run: npm i # ci (not install) to install exact versions from package-lock.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this comment or fix the fact that it is false?
|
I currently still see the issue of not being able to follow the about-us link. Dockerfile run the container go to There should be a file called The issue does not have to do with the type of static file server because both the python and the npx static file server give a 404 |
|
Also it has been really hard to figure out how to run the tests locally, I still have not yet. Can you add a github action that runs the tests so we dont have to rely on our own machine? |
|
Just tried again, and I am not getting the about-us not loading issue, but me and Ted seem to have found another issue |
|
Run a static file server on port 5173 Then in a different terminal run the tests The donate test fails. You can verify this by going to the donate page directly |
I updated the GH action (
./.github/workflows/static.yml) to allow manually kicking off a build, and disabled automatic builds with pushes to main.npm cifor some reason wouldn't update thepackage-lock.jsonand kept complaining that it was out of sync even after runningnpm ilocally to update the lock file. The action completed successfully after changing tonpm i.https://github.com/kevinkace/Local-Connectivity-Lab.github.io/actions/runs/20872694540
This should be changed back to
npm ciafter the build completes successfully.Lastly, I removed specifying
#eachkeys, and updated./estlint.config.jsto suit. Keys are only required if items are added/removed from an#each, which currently we're not doing.