Skip to content

Conversation

@itsme-zeix
Copy link
Contributor

Description

Add support for Rubrehaku

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

  • None

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@itsme-zeix itsme-zeix marked this pull request as draft February 9, 2025 05:28
@itsme-zeix itsme-zeix marked this pull request as ready for review February 9, 2025 05:45
@dunk2k
Copy link
Contributor

dunk2k commented Feb 10, 2025

@itsme-zeix
In layout LAYOUT_all, when compared to the other layouts is the missing top right key intentional?

@dunk2k
Copy link
Contributor

dunk2k commented Feb 10, 2025

@itsme-zeix In layout LAYOUT_all, when compared to the other layouts is the missing top right key intentional?

Or should it be that all layouts don't have a top right key? as shown in matrix_diagram.md

If correct, then community_layouts should be removed

@itsme-zeix
Copy link
Contributor Author

itsme-zeix commented Feb 10, 2025

@itsme-zeix In layout LAYOUT_all, when compared to the other layouts is the missing top right key intentional?

Or should it be that all layouts don't have a top right key? as shown in matrix_diagram.md

If correct, then community_layouts should be removed

@dunk2k Thank you. Appreciate the quick review.
All layouts should not have a top right key - though the functionality is identical with or without it. Removing community_layouts makes sense, no wonder I was having compilation issues.

@github-actions
Copy link

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with bug, awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

@github-actions github-actions bot added the stale Issues or pull requests that have become inactive without resolution. label Mar 28, 2025
@itsme-zeix
Copy link
Contributor Author

itsme-zeix commented Mar 28, 2025

Doesn’t seem like I can change the stale label :(

@github-actions github-actions bot removed the stale Issues or pull requests that have become inactive without resolution. label Mar 29, 2025
@itsme-zeix
Copy link
Contributor Author

@waffle87 Thank you for the review! I have committed the relevant changes that you have suggested.

@itsme-zeix itsme-zeix requested a review from waffle87 May 7, 2025 18:44
@itsme-zeix itsme-zeix closed this May 7, 2025
@itsme-zeix itsme-zeix reopened this May 7, 2025
@waffle87 waffle87 requested a review from a team May 7, 2025 18:50
@github-actions
Copy link

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with bug, awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

@github-actions github-actions bot added the stale Issues or pull requests that have become inactive without resolution. label Jun 22, 2025
@itsme-zeix
Copy link
Contributor Author

Bumping this PR for another review and merge.

@github-actions github-actions bot removed the stale Issues or pull requests that have become inactive without resolution. label Jun 23, 2025
@waffle87 waffle87 requested a review from a team June 27, 2025 19:45
@github-actions
Copy link

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with bug, awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

@github-actions github-actions bot added the stale Issues or pull requests that have become inactive without resolution. label Aug 21, 2025
@github-actions
Copy link

Thank you for your contribution!
This pull request has been automatically closed because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, or re-open when it's ready.
// [stale-action-closed]

@github-actions github-actions bot closed this Sep 21, 2025
@waffle87 waffle87 reopened this Oct 25, 2025
@waffle87 waffle87 mentioned this pull request Oct 25, 2025
14 tasks
@github-actions github-actions bot added core documentation CI dd Data Driven Changes and removed stale Issues or pull requests that have become inactive without resolution. labels Oct 25, 2025
@itsme-zeix itsme-zeix changed the title Add Rubrehaku [Keyboard] Add Rubrehaku Oct 26, 2025
@itsme-zeix itsme-zeix requested a review from zvecr October 26, 2025 14:17
@zvecr zvecr removed core documentation CI dd Data Driven Changes labels Oct 26, 2025
@zvecr zvecr merged commit 024c4ef into qmk:master Nov 8, 2025
5 checks passed
coldwateryu pushed a commit to coldwateryu/qmk_firmware that referenced this pull request Nov 24, 2025
cadusk pushed a commit to cadusk/qmk_firmware that referenced this pull request Nov 29, 2025
* upstream/master: (24 commits)
  Bump actions/checkout from 4 to 6 (qmk#25829)
  QMK CLI Environment bootstrapper (qmk#25038)
  Add Demod LM Rev. 1 (qmk#25793)
  Add support for Coffee Break Keyboards' Coffeevan (qmk#25805)
  [Keyboard] Add Keenome Keys' "The Grid v2" (qmk#25813)
  Bump actions/checkout from 5 to 6 (qmk#25807)
  [Keyboard] Add Coffee Break Keyboards Acai (qmk#25796)
  Short term fix for skip_converter KeyError in 'qmk userspace-add' (qmk#25798)
  Hyper7 v4 (qmk#25728)
  Add imi60-HS (qmk#25773)
  Fix detection of hid bootloader flashing tool (qmk#25790)
  VIA Keylog Change (qmk#25504)
  [Bugfix] QP error handling (qmk#25591)
  [Keyboard] Add Rubrehaku (qmk#24907)
  Fix pmw33xx sensor initialisation (qmk#25777)
  Add LED index map to `qmk info` cli command (qmk#25743)
  Bump actions/download-artifact from 5 to 6 (qmk#25746)
  Bump actions/upload-artifact from 4 to 5 (qmk#25745)
  Bump JamesIves/github-pages-deploy-action from 4.7.3 to 4.7.4 (qmk#25771)
  Deprecate LAYOUT() macro in favor of JSON matrix definitions
  ...
eth4nH pushed a commit to eth4nH/yunzii_firmware that referenced this pull request Dec 4, 2025
versality pushed a commit to versality/qmk_firmware that referenced this pull request Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants