Skip to content

Conversation

@mattieweathersby
Copy link
Contributor

BOYYYYY!!!!! This was difficult!!! I could not for the life of me grasp what I was doing because seeing the letters in place for some reason made me feel like I couldn't figure it out unless there were numbers to substitute it. But after the help from some fellow comrades, I was able to grasp what was going on. I would definitely like to do something like this again so the information can stick.

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: Mattie Weathersby
Date: 8/19/2025

Feedback

Mattie Weathersby successfully met all the functional requirements for the assignment. The calculate method in expression_calculator.ts was implemented correctly, using helper methods for addition, subtraction, multiplication, division, and power operations. This modular approach enhances readability and maintainability. The .env.test file was also correctly configured with the assigned group letter 'D', ensuring compliance with the assignment's requirements. From a technical perspective, the code is well-structured and adheres to TypeScript standards, with consistent style and no syntax errors. All GitHub checks passed, indicating the code is robust and free of issues. However, there was no attempt to tackle the stretch assignment, resulting in a stretch score of 0. Overall, excellent work on the main assignment, with room for further exploration in the stretch tasks.


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.

Well done, just what I asked for.

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