-
Notifications
You must be signed in to change notification settings - Fork 427
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
CI job for live counting active participation #2459
Comments
I'm at home sick, so would like to work on this. Is it a race for who submits the first PR? 😜 |
Hi @bjornthiberg ! You won the race of taking up this issue :) It is all yours. Looking forward to a PR soon :D |
@bjornthiberg Could you please submit the pull request for this task like you do for presentation, demo, etc? We need that to enumerate tasks later. |
algomaster99
pushed a commit
that referenced
this issue
Sep 12, 2024
* feat: add lecture participation tracker using GitHub Actions * use issue number instead of title * use newest versions of checkout and setup-python, pin python 3.11.8 * set timezone in script instead of workflow file * make default behavior be print in plaintext * Move configuration to external JSON file * fix: use correct file path for config * remove task proposal * add required yearly updates to documentation * separate commenting duration into lecture and leeway durations * update workflow to run in tools dir, get config from current dir * fix: use timezone in "last changed" timestamps
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi all,
We are back with another idea for an open-source contribution.
This year we introduced active participation as one of the criteria for grades. Now, it's a CI course. We won't be counting who asked question in what lecture, right? Let's write a CI script in this repo and whoever does this will get an open source contribution task. Some checks we need to have:
Let's start with these criteria and maybe we evolve based on the different comments we see. :D
@monperrus @sofiabobadilla @javierron @Deee92 Do you want to add more factors above?
The text was updated successfully, but these errors were encountered: