West Midlands | July-SDC | Gabriel Deng |Sprint 4 | Laptop-allocation

[gai93003]

Learners, PR Template

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Module-Decomposition - Sprint 4) doesn't match expected format (example: 'Sprint 2', without quotes)

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Module-Decomposition - Sprint 4) doesn't match expected format (example: 'Sprint 2', without quotes)

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Module-Decomposition - Sprint 4) doesn't match expected format (example: 'Sprint 2', without quotes)

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Module-Decomposition - Sprint 4) doesn't match expected format (example: 'Sprint 2', without quotes)

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Module-Decomposition - Sprint 4) doesn't match expected format (example: 'Sprint 2', without quotes)

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Module-Decomposition - Sprint 4) doesn't match expected format (example: 'Sprint 2', without quotes)

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Module-Decomposition - Sprint 4) doesn't match expected format (example: 'Sprint 2', without quotes)

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[github-actions]

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[LonMcGregor]

This is a good start at this task, and it will work for the interactive method you're using. But if you re-read the spec in the task (https://sdc.codeyourfuture.io/decomposition/sprints/4/backlog/#04-implement-laptop-allocation) It's asking for a specific method that takes a list of possible laptops, a list of possible people, and then returns a matching for all of them.

Can you extend your code to meet that criteria?

[gai93003]

Hey @LonMcGregor, thanks for clearing that up for me. I will try to adapt my current implementation to meet that criteria.

…

On Tue, 28 Oct 2025 at 15:15, LonMcGregor ***@***.***> wrote: ***@***.**** commented on this pull request. This is a good start at this task, and it will work for the interactive method you're using. But if you re-read the spec in the task ( https://sdc.codeyourfuture.io/decomposition/sprints/4/backlog/#04-implement-laptop-allocation) It's asking for a specific method that takes a list of possible laptops, a list of possible people, and then returns a matching for all of them. Can you extend your code to meet that criteria? — Reply to this email directly, view it on GitHub <#38 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BC3JQE6W22B2EQAS5FAYLOD3Z6B7RAVCNFSM6AAAAACJVO2WI2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTGOBZGMYDENBQGY> . You are receiving this because you modified the open/close state.Message ID: ***@***.*** com>