-
-
Notifications
You must be signed in to change notification settings - Fork 116
Add requirement to pass a Codility test #1566
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
base: main
Are you sure you want to change the base?
Conversation
Rather than having this be an entry criterion to the Piscine, we want to make it an exit criterion of ITP. This better reflects completing ITP meaning you can apply the skills learnt in ITP.
✅ Deploy Preview for cyf-launch ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-common ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-sdc ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-programming ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-curriculum ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-tracks ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-piscine ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-itd ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Are we starting this requirement with the current ITP cohort or the Jan-26 ITP? |
|
We have not decided if we need to change this, right, @illicitonion ? |
|
Yeah, not yet decided - so definitely not the current cohort, but maybe the next one. |
PaulBrytonRaj18
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Approving:
Good addition. The Codility test requirement is clearly documented with proper links.
This effectively shifts the requirement from entry to exit criterion, which aligns well
with validating learner skills after ITP completion.
One note: Make sure the linked resources (issue #321) are up-to-date with any recent changes.

Rather than having this be an entry criterion to the Piscine, we want to make it an exit criterion of ITP. This better reflects completing ITP meaning you can apply the skills learnt in ITP.