Skip to content
This repository has been archived by the owner on May 30, 2023. It is now read-only.

Add Practise Exercise Largest Series Product. #74

Closed
wants to merge 5 commits into from
Closed

Add Practise Exercise Largest Series Product. #74

wants to merge 5 commits into from

Conversation

LeaveNhA
Copy link
Member

@LeaveNhA LeaveNhA commented Sep 5, 2021

Yet another practise exercise.

@LeaveNhA LeaveNhA added the x:module/practice-exercise Work on Practice Exercises label Sep 5, 2021
@LeaveNhA LeaveNhA self-assigned this Sep 5, 2021
ErikSchierboom
ErikSchierboom reviewed Sep 8, 2021
View reviewed changes
Copy link
Member

@ErikSchierboom ErikSchierboom left a comment

Choose a reason for hiding this comment

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

LGTM. There are two exercises in this PR though, as the dominoes exercise is also included (both are good).

The only thing that is missing is an entry for each exercise in the root config.json file.

@LeaveNhA
Copy link
Member Author

LeaveNhA commented Sep 11, 2021 •
edited

I've added.

Edit: there was a conflict in config file. I've solved it. Thanks, @ErikSchierboom! 😇

@LeaveNhA
Copy link
Member Author

I've gather (accidentaly) all exercises together in a new PR.

@LeaveNhA LeaveNhA closed this Sep 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ErikSchierboom ErikSchierboom Awaiting requested review from ErikSchierboom

Assignees

@LeaveNhA LeaveNhA

Labels
x:module/practice-exercise Work on Practice Exercises
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@LeaveNhA @ErikSchierboom