-
Notifications
You must be signed in to change notification settings - Fork 29
feat: adds lesson_16 homework and lesson_17 pre-work #542
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
Conversation
Signed-off-by: Anthony D. Mays <[email protected]>
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.
Pull Request Overview
This PR adds lesson 16 homework content and lesson 17 pre-work materials for an object-oriented programming course. The changes introduce assignment instructions for custom data types and set up the initial structure for lesson 17.
- Adds detailed homework assignment for lesson 16 focused on creating custom data types with specific requirements
- Creates lesson 17 pre-work README with video and reading resources
- Sets up a complete Gradle project structure for lesson 16 objects assignment
Reviewed Changes
Copilot reviewed 11 out of 12 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| lesson_16/README.md | Added comprehensive homework assignment for custom data types with specific requirements |
| lesson_17/README.md | Created pre-work materials including video tutorials and articles on OOP and naming conventions |
| lesson_16/objects/... | Generated complete Gradle project structure with Spring Boot configuration for the homework assignment |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
||
| ## Homework | ||
|
|
||
| - TODO(anthonydmays): Add some details l8tr. No newline at end of file |
Copilot
AI
Sep 10, 2025
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.
The abbreviation 'l8tr' should be 'later' for proper documentation.
| - TODO(anthonydmays): Add some details l8tr. | |
| - TODO(anthonydmays): Add some details later. |
| jacoco | ||
| id("io.freefair.lombok") version "8.10.2" | ||
| id("com.diffplug.spotless") version "6.25.0" | ||
| id("org.springframework.boot") version "3.4.0" |
Copilot
AI
Sep 10, 2025
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.
Mixed indentation detected. This line uses tabs while other lines use spaces. Should use consistent spacing throughout the file.
| id("org.springframework.boot") version "3.4.0" | |
| id("org.springframework.boot") version "3.4.0" |
| } | ||
|
|
||
| tasks.check { | ||
| dependsOn(tasks.jacocoTestCoverageVerification) |
Copilot
AI
Sep 10, 2025
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.
Mixed indentation detected. This line uses tabs while other lines use spaces. Should use consistent spacing throughout the file.
| dependsOn(tasks.jacocoTestCoverageVerification) | |
| dependsOn(tasks.jacocoTestCoverageVerification) |
Signed-off-by: Anthony D. Mays [email protected]