Skip to content

fix: typo in changelog header link#41

Merged
Julusian merged 1 commit intomainfrom
Julusian-patch-1
Mar 3, 2026
Merged

fix: typo in changelog header link#41
Julusian merged 1 commit intomainfrom
Julusian-patch-1

Conversation

@Julusian
Copy link
Member

@Julusian Julusian commented Mar 2, 2026

About the Contributor

This pull request is posted on behalf of the BBC

Type of Contribution

This is a: Bug fix

New Behavior

Fix a typo

Testing Instructions

Other Information

Status

  • PR is ready to be reviewed.
  • The functionality has been tested by the author.
  • Relevant unit tests has been added / updated.
  • Relevant documentation (code comments, system documentation) has been added / updated.

Summary

Fixed a typo in the HEADER constant in bin/updateVersion.mjs (line 38), changing "Convential Commits" to "Conventional Commits" in the changelog header link.

Changes

bin/updateVersion.mjs

  • Line 38: Corrected the text in the HEADER constant from "Convential Commits" to "Conventional Commits"

This fixes the typo that was being written into the CHANGELOG.md file header whenever the version update script runs.

@coderabbitai
Copy link

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 84823e6 and a240d21.

📒 Files selected for processing (1)
  • bin/updateVersion.mjs

Walkthrough

Fixed a typo in the HEADER constant within the version update script, changing "Convential Commits" to "Conventional Commits". No functional or control flow changes were made.

Changes

Cohort / File(s) Summary
Typo Fix
bin/updateVersion.mjs
Corrected spelling error in HEADER constant: "Convential Commits" → "Conventional Commits"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

🐰 A letter lost, now found again,
In headers where commits are pen,
"Conventional" hops in place,
With spelling set at proper pace! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly describes the main change: fixing a typo in the changelog header link, which matches the actual modification in the code.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch Julusian-patch-1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Julusian Julusian merged commit 451c030 into main Mar 3, 2026
5 checks passed
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