Skip to content

Dead Code Backlog Exercises#332

Merged
Poonam-raj merged 3 commits intomainfrom
praj-deadcode-backlog
Dec 2, 2025
Merged

Dead Code Backlog Exercises#332
Poonam-raj merged 3 commits intomainfrom
praj-deadcode-backlog

Conversation

@Poonam-raj
Copy link
Copy Markdown
Contributor

Issue created here: #331

Relating to the Prep material, pull request here: CodeYourFuture/curriculum#1669

Exercises aim to challenge students to identify dead code in an existing code base.

Labels I will add to the backlog issue once this backlog work has been approved will be
image

Please let me know if the complexity of this work feels appropriate. I believe the size will be "small".

@github-actions
Copy link
Copy Markdown

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

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

Reason: Wrong number of parts separated by |s

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).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

@Poonam-raj Poonam-raj added the NotCoursework A PR that edits material in the module. Needs review by the curriculum team. label Nov 28, 2025
Copy link
Copy Markdown

@LonMcGregor LonMcGregor left a comment

Choose a reason for hiding this comment

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

The work code in these repositories is usually styled to have semis at the end of lines;

Is it intentional that you leave these off here?

Other than that, this looks good to go.

@Poonam-raj
Copy link
Copy Markdown
Contributor Author

The work code in these repositories is usually styled to have semis at the end of lines;

Is it intentional that you leave these off here?

Other than that, this looks good to go.

Not intentional, will amend

Copy link
Copy Markdown

@LonMcGregor LonMcGregor left a comment

Choose a reason for hiding this comment

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

Looks good now

@Poonam-raj Poonam-raj merged commit c89c066 into main Dec 2, 2025
@Poonam-raj Poonam-raj deleted the praj-deadcode-backlog branch December 2, 2025 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NotCoursework A PR that edits material in the module. Needs review by the curriculum team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants