Skip to content

Conversation

@Konamiman
Copy link
Owner

@Konamiman Konamiman commented Nov 16, 2025

Update kernel version number to 2.1.4

Summary by CodeRabbit

  • Chores
    • Bumped kernel version to 2.1.4.
    • Updated internal version constants for DOS 2.50 compatibility.

@Konamiman Konamiman added this to the v2.1.4 milestone Nov 16, 2025
@Konamiman Konamiman self-assigned this Nov 16, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 16, 2025

Walkthrough

Two version constants are incremented: the build system VERSION is bumped from 2.1.3 to 2.1.4, and the DOS 2.50 SEC_NEXTOR_VERSION_LOW constant is updated from 3 to 4. No logic or control-flow modifications occur.

Changes

Cohort / File(s) Summary
Version constants
source/kernel/Makefile, source/kernel/kvar.mac
Increment build VERSION from 2.1.3 to 2.1.4; update SEC_NEXTOR_VERSION_LOW constant from 3 to 4 in DOS 2.50 section

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop and a bump, version numbers now dance,
From 2.1.3 to 2.1.4 with a gleeful prance!
And DOS 2.50 counts a bit higher too,
Small changes that keep the kernel feeling brand new! ✨

Pre-merge checks and finishing touches

✅ 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 accurately and concisely describes the main change in the pull request - bumping the kernel version to 2.1.4 as confirmed by both Makefile and kvar.mac changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update/version-numbers-2.1.4

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eeca10e and db846c4.

📒 Files selected for processing (2)
  • source/kernel/Makefile (1 hunks)
  • source/kernel/kvar.mac (1 hunks)
🔇 Additional comments (2)
source/kernel/Makefile (1)

11-11: LGTM! Version bump is correct.

The VERSION variable update from 2.1.3 to 2.1.4 correctly reflects the patch version increment and will be used consistently throughout the build system to name the output ROM files.

source/kernel/kvar.mac (1)

29-29: LGTM! Version constant synchronized with Makefile.

The SEC_NEXTOR_VERSION_LOW constant update from 3 to 4 correctly aligns with the Makefile VERSION change, maintaining consistency between the build system version (2.1.4) and the runtime kernel version constant.


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.

@Konamiman Konamiman changed the base branch from v2.1 to v2.1.4 November 16, 2025 15:53
@Konamiman Konamiman merged commit 7310f07 into v2.1.4 Nov 16, 2025
1 of 2 checks passed
@Konamiman Konamiman mentioned this pull request Nov 16, 2025
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