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

Course automation Proposal: Checking legal group composition #996

Merged
merged 1 commit into from
Mar 28, 2021
Merged

Course automation Proposal: Checking legal group composition #996

merged 1 commit into from
Mar 28, 2021

Conversation

EleonoraBorzi
Copy link

Course Automation Proposal: Checking legal group composition

Members:

Henrik Kultala (kultala@kth.se)
GitHub: hengque

Eleonora Borzi (borzi@kth.se)
GitHub: EleonoraBorzis

Group rules for this course:

  • Max 3 persons in a group

  • You cannot be with the same persons for more than 2 projects.

  • You can work alone on one or at most two projects.

  • When you send a pull request for registration, please follow the name convention of using the email addresses of two members to create the folder: email-email.

    • This is assumed to be the same email addresses as specified in the readme.

    • This is also assumed to mean “all members of the group” and not “two members”; i.e. if there are three members then all three members email address should be included in the folder name.

Proposal

When a new project is proposed, the readme and the folder name are checked for the names of the group members. The group validity is checked against the rules stated above and a comment with the results is posted on the PR. It also checks that the name of the folder is correct in relation to the email addresses in the readme.

Tools:

Github Actions
Bonus: Status Check on GitHub

@SophieHYe SophieHYe self-assigned this Mar 28, 2021
@SophieHYe SophieHYe self-requested a review March 28, 2021 13:31
@SophieHYe
Copy link

Hi,

Your proposal looks great! Look forward to your contribution on the course automation! I am merging it now.

@SophieHYe SophieHYe merged commit 22d79fa into KTH:2021 Mar 28, 2021
arxra added a commit to arxra/devops-course that referenced this pull request Mar 29, 2021
* '2021' of github.com:KTH/devops-course: (52 commits)
  doc: Course automation proposal (KTH#1008)
  Adds essay proposal (KTH#1009)
  Create readme.md (KTH#996)
  Create README.md (KTH#995)
  doc: add essay proposal for devops course (KTH#988)
  Presentation Proposal: Continuous testing and its role & application within devops (KTH#1026)
  Executable tutorial: nginx on vagrant box (KTH#956)
  Presentation week2 (KTH#1005)
  Course automation: Verify pull request content changes (KTH#981)
  Presentation: The history of devops (KTH#1004)
  Presentation : Influence of DevOps in Software testing (KTH#984)
  Course automation: Automatic validation and labeling (KTH#949)
  Essay: Automated tests for infrastructure code (KTH#999)
  Demo: How to add code coverage report on Github using Clover and GitHub Action (KTH#994)
  Demo proposal: Implementation of Docker to React Application (KTH#992)
  Course Automation: automatic labels (KTH#932)
  Update README.md
  Reschedule Nix presentation to week 2 (KTH#1001)
  Course Automation: Upvote other students' work (KTH#991)
  Presentation Kanban vs Scrum (KTH#993)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants