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

Switch to codespaces and remove gitpod support #69

Merged
merged 4 commits into from
Feb 7, 2024

Conversation

BacLuc
Copy link
Contributor

@BacLuc BacLuc commented Feb 7, 2024

No description provided.

The image got too large with the document compilation, so we remove it.
That it also supports systems without the service abstraction layer.
@BacLuc BacLuc requested a review from langchr86 February 7, 2024 10:30
topics/linux/linux_exercise.md Outdated Show resolved Hide resolved
topics/linux/linux_exercise.md Outdated Show resolved Hide resolved
topics/linux/linux_exercise.md Outdated Show resolved Hide resolved
topics/linux/linux_exercise.md Outdated Show resolved Hide resolved
.github/workflows/build-images.yml Outdated Show resolved Hide resolved
The tail -f /dev/null is needed for the java service,
else the devcontainer/java stops and then the codespace creation fails.
Then the users only have to pull the image, and we test that the image can be built.
The action build-images retrieves the build information from all docker-compose.yml files
in the repo and builds the images in parallel.
It also handles the correct tagging (commit hash, nightly on main, latest on tag, tag on tag)
of the image.
Adapted from: https://github.com/ase-23-the-bald-owls/swiss-card-exchange/blob/462c0b083dffad7e5275b7a8e4544406fc2527d5/.github/workflows/build-images.yml
@BacLuc BacLuc force-pushed the switch-to-codespaces branch from 7852f2b to 9f09dae Compare February 7, 2024 13:13
@langchr86 langchr86 merged commit bc184c6 into scs:main Feb 7, 2024
5 checks passed
@BacLuc BacLuc deleted the switch-to-codespaces branch April 5, 2024 07:59
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