Skip to content

Conversation

@StefanJum
Copy link

On every push on the main branch, check if any files related to a lab
have changed. If so, create an archive with all files related to that
lab and push it to the lab-archives branch.

The gen-zip.py script will parse the config.yaml file, check the
last commit, and if the commit changes any file related to one of the
labs, extract all files related to that lab and create a zip archive.

On every push on the main branch, check if any files related to a lab
have changed. If so, create an archive with all files related to that
lab and push it to the `lab-archives` branch.

The `gen-zip.py` script will parse the `config.yaml` file, check the
last commit, and if the commit changes any file related to one of the
labs, extract all files related to that lab and create a zip archive.

Signed-off-by: Stefan Jumarea <[email protected]>
@StefanJum StefanJum force-pushed the StefanJum/archive-labs branch from e60fc9b to 3d9ffb9 Compare September 29, 2025 16:40
Copy link

@teodutu teodutu left a comment

Choose a reason for hiding this comment

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

So students will just git checkout lab-archives and unzip the files from there? That should work.

I believe ChatGPT did a good enough job.

@StefanJum
Copy link
Author

Even better, we can give them the links to the archives in the lab text, and they can download it without using git at all.

@teodutu teodutu merged commit 20a19a0 into cs-pub-ro:main Oct 1, 2025
3 checks passed
@teodutu
Copy link

teodutu commented Oct 2, 2025

I forgot to mention this before merging this PR, but now that we're giving students one zip file per lab, it would make sense to delete this section: https://github.com/cs-pub-ro/operating-systems/blob/main/misc/lab-setup.md#save-progress-and-prepare-next-lab.

@StefanJum
Copy link
Author

I'm working on a PR that will add links to the labs and modify instructions where relevant. I'll let you know when it's ready for review
#169

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