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.

Sign up for GitHub

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

Jump to bottom

Ticket-125D #164

Closed
wants to merge 4 commits into from
Closed

Ticket-125D #164

wants to merge 4 commits into from

Conversation

kazeez01
Copy link

  • Created new file called 18 Team members.md
  • Added my name to this file
  • Added my class name to this file

@ziobron
Copy link
Contributor

ziobron commented Feb 24, 2024

@kazeez01, we do not have your data in the Coders School database.

  1. Please fill in this form https://forms.gle/rkzLsMUSBdnGQgdM9
  2. Edit this PR to run this automation once again (you can just add a space in the description or title).
  3. Wait one minute. If any label was added to this PR (like Standard, Premium, Podstawy C++, STL) then everything is fine :)
  4. If your solution passed, but nothing happens, modify your code (eg. add an empty line) to run the workflow again.

If you don't know what to do - ask on Discord and provide a link to this PR.

lioneltchami
lioneltchami suggested changes Feb 24, 2024
View reviewed changes
Copy link

@lioneltchami lioneltchami left a comment

Choose a reason for hiding this comment

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

Good job @kazeez01 but please fix the comments

18-team-members.md Outdated
# My name: Kehinde A


## CLass name: DevSecOps 2024
Copy link

Choose a reason for hiding this comment

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

Please spell again ... Class instead of CLass

18-team-members.md
@@ -0,0 +1,4 @@
# My name: Kehinde A
Copy link

Choose a reason for hiding this comment

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

Well done but we want full names please

@ziobron
Copy link
Contributor

ziobron commented Feb 24, 2024

@kazeez01, we do not have your data in the Coders School database.

  1. Please fill in this form https://forms.gle/rkzLsMUSBdnGQgdM9
  2. Edit this PR to run this automation once again (you can just add a space in the description or title).
  3. Wait one minute. If any label was added to this PR (like Standard, Premium, Podstawy C++, STL) then everything is fine :)
  4. If your solution passed, but nothing happens, modify your code (eg. add an empty line) to run the workflow again.

If you don't know what to do - ask on Discord and provide a link to this PR.

@ziobron
Copy link
Contributor

ziobron commented Mar 3, 2024

@kazeez01 Your solution has not passed all checks (maybe it was only a formatting-check?) and you haven't modified this Pull Request during the last 7 days. It will be automatically closed after the next 7 days if it remains untouched.

1 similar comment
@ziobron
Copy link
Contributor

ziobron commented Mar 10, 2024

@kazeez01 Your solution has not passed all checks (maybe it was only a formatting-check?) and you haven't modified this Pull Request during the last 7 days. It will be automatically closed after the next 7 days if it remains untouched.

@ziobron
Copy link
Contributor

ziobron commented Mar 17, 2024

We are closing this PR. If you want to fix your code and get points, please re-open it or raise a new PR

@ziobron ziobron closed this Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lioneltchami lioneltchami lioneltchami requested changes

Assignees
No one assigned
Labels
Auto-closed Unknown user
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@kazeez01 @ziobron @lioneltchami @arafalski