Skip to content

fix(curriculum): correct code fence for merge sort example#68317

Open
Yaswanthram-396 wants to merge 2 commits into
freeCodeCamp:mainfrom
Yaswanthram-396:fix-merge-sort-code-fence
Open

fix(curriculum): correct code fence for merge sort example#68317
Yaswanthram-396 wants to merge 2 commits into
freeCodeCamp:mainfrom
Yaswanthram-396:fix-merge-sort-code-fence

Conversation

@Yaswanthram-396

Copy link
Copy Markdown

Checklist:

Closes #68286

Description

This PR updates the merge sort lesson by changing the code fence for the list-state example from javascript to md, since the block is a visual illustration rather than executable JavaScript. This ensures consistent formatting with the surrounding examples and avoids incorrect syntax highlighting.

@Yaswanthram-396 Yaswanthram-396 requested review from a team as code owners June 27, 2026 20:05
@github-actions github-actions Bot added scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. deprioritized PR reviews are deprioritized; lacks productive input and ignores codebase best practices. labels Jun 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hi there,

Thanks for opening this pull request.

The automated checks found some issues:

Checklist: The PR description is missing the required checklist or some of its items are not completed:

  1. The Checklist: heading is present in the PR description.
  2. The checkbox items are ticked (changed from [ ] to [x]).
  3. You have actually completed the items in the checklist.

Please edit your PR description to include the following template with the checklist items completed.

Checklist:

<!-- Please follow this checklist and put an x in each of the boxes, like this: [x]. It will ensure that our team takes your pull request seriously. -->

- [ ] I have read and followed the [contribution guidelines](https://contribute.freecodecamp.org).
- [ ] I have read and followed the [how to open a pull request guide](https://contribute.freecodecamp.org/how-to-open-a-pull-request/).
- [ ] My pull request targets the `main` branch of freeCodeCamp.
- [ ] I have tested these changes either locally on my machine, or GitHub Codespaces.

<!--If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.-->

Closes #XXXXX

<!-- Feel free to add any additional description of changes below this line -->

Join us in our chat room or our forum if you have any questions or need help with contributing.


// merge the lists in sorted order
17 53
```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

there is no reason to remove the closing fence, please restore

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks! I've restored the closing code fence while keeping the code fence language change.

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

Labels

deprioritized PR reviews are deprioritized; lacks productive input and ignores codebase best practices. scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect code fence language and awkward phrasing in the merge sort lecture

2 participants