Skip to content

Conversation

@thusser252
Copy link
Contributor

No description provided.

…orial, getFirstNFibonacciNumbers, binarySearch, and other conditional functions
Copy link
Contributor

@anthonydmays anthonydmays left a comment

Choose a reason for hiding this comment

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

🎓 Automated Grading Report

Student: Trishtan Husser
Date: 8/22/2025

Feedback

Trishtan Husser successfully implemented all the required functions in both lesson7.ts and part_c.ts files, meeting the functional requirements for the assignment. The implementations for compareStrings, computeFactorial, getFirstNFibonacciNumbers, binarySearch, isStrongPassword, getDayOfWeek, and getTicketPrice are correct and demonstrate a good understanding of the problem statements. However, the technical score is impacted by the failing GitHub checks, indicating potential issues with code style, TypeScript syntax, or other technical requirements. It's important to ensure all checks pass before submission to demonstrate adherence to technical standards. Additionally, no stretch criteria were provided or attempted, so the stretch score is not applicable. For future submissions, focus on resolving any build issues and ensuring all automated checks pass to improve the technical score.


This is an automated preliminary review. Please review and adjust before finalizing.

Copy link
Contributor

@anthonydmays anthonydmays left a comment

Choose a reason for hiding this comment

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

You need to include the modified .env.test file as well so that the tests will run for part_c.

*/
export function computeFactorial(n: number): number {
return 0;
if (n <= 1) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Your tests are not passing, please fix your computeFactorial implementation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants