Skip to content
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

Amrith new branch #16

Open
wants to merge 65 commits into
base: main
Choose a base branch
from
Open

Conversation

amriith
Copy link

@amriith amriith commented Sep 12, 2024

No description provided.

Provide the option to build containers via `docker compose build` for
cases where the configuraiton is being changed.
Ensure that dev containers can be versioned. Provide instructions on the
process.
Add a dev-container capable of building, testing, and running all
required components for the basic application.
- provide redis in the dev container
- update bundler to 2.4
- move gems from root user to vscode user
When the database is missing, the dev container will create the
database, populate it with students, and simulate sign off.
- ensure database launches
- create database when not present
- retain zsh history for the project
- add docker compose for testing devcontainer
- add build script to create devcontainer image
- add husky to root project
Ensure that database is created and user can see details.
macite and others added 30 commits March 27, 2023 20:53
This should fix issues with this working on Windows.
- clear out database in image
- setup student-work and workspace tmp
- add bundle install for api
- add npm install for web

This ensures that these are updated to match the
bundle and package.
Added update to explain launch via Dev Container

Daniel Maddern
…structions

docs: contributing-dev-containers
- ensure student work is always vscode
- ensure simnulate happens... and doesn't end
fix: force npm install in the vscode task
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.

5 participants