-
Notifications
You must be signed in to change notification settings - Fork 12
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
Feature/68 migration to null safety #939
Feature/68 migration to null safety #939
Conversation
…com/ApplETS/Notre-Dame into feature/68-migration-to-null-safety
…com/ApplETS/Notre-Dame into feature/68-migration-to-null-safety
test/ui/views/goldenFiles/gradesDetailsView_evaluation_not_completed.png
Outdated
Show resolved
Hide resolved
…ety' into feature/68-migration-to-null-safety
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.
LGTM! Good job Xavier, t'as bien fait ça, null safety nous voilà!!!😄 Merci à @apomalyn pour l'aide aussi!
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.
LGTM! Good job!!!! 😁
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.
Autres que la question en haut LGTM! 😄
…com/ApplETS/Notre-Dame into feature/68-migration-to-null-safety
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Closes #68
Null safety
📖 Description
Migrate to null safety
🧪 How Has This Been Tested?
Everything was tested
☑️ Checklist before requesting a review
version: Major
,version: Minor
orversion: Patch
.