Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assume only 32u4 boards can run 'promicro' converters #23312

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

zvecr
Copy link
Member

@zvecr zvecr commented Mar 19, 2024

Description

Users should not be trying to target "blackpill" based boards with the current available converters.

The following is logically incorrect...

qmk compile -kb cantor -km default -e CONVERT_TO=promicro_rp2040

However the error should be a bit more human readable than what is produced.

Long term, #20330 will solve the issue but that involves many keyboard upates.

Types of Changes

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

Issues Fixed or Closed by This PR

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

@github-actions github-actions bot added the core label Mar 19, 2024
@zvecr zvecr requested a review from a team March 19, 2024 23:50
@tzarc
Copy link
Member

tzarc commented Mar 20, 2024

Should this go to master?

@zvecr zvecr mentioned this pull request Mar 20, 2024
14 tasks
@zvecr zvecr force-pushed the converter_promicro_only_32u4 branch from 55c6b8f to 6596271 Compare March 20, 2024 19:52
@zvecr zvecr changed the base branch from develop to master March 20, 2024 19:52
@zvecr zvecr merged commit 34374d2 into master Mar 20, 2024
2 of 3 checks passed
@zvecr zvecr deleted the converter_promicro_only_32u4 branch March 20, 2024 22:24
dlip added a commit to dlip/qmk_firmware that referenced this pull request Mar 21, 2024
* upstream/master: (40 commits)
  [Keyboard] Add marshkeys flowerpad (qmk#23240)
  Add TechnicPad macropad (qmk#23238)
  Add handwired/Jotlily60 (qmk#23212)
  Add the EQUATOR keyboard (qmk#23237)
  Support for slice65 from PizzaKeyboards (qmk#23148)
  Add Swiss support (qmk#23272)
  Assume only 32u4 boards can run 'promicro' converters (qmk#23312)
  Add Macropad 6Pad (qmk#23099)
  Fix invalid use of stemcell converter (qmk#23317)
  Updates to PR Checklist (qmk#23318)
  [Keyboard] Add posey_split v4/v5 (qmk#23258)
  Added WT65-H3 (HIBI HIBIKI) (qmk#22886)
  Added WT65-G3 (HIBI HIBIKI) (qmk#22888)
  Modify wording in 'getting_started_introduction.md' (qmk#23232)
  Bump tj-actions/changed-files from 42 to 43 (qmk#23282)
  AMJKeyboard AMJ96 Layout Additions (qmk#23267)
  Fixup cipulot eeprom. (qmk#23280)
  Update feature_tri_layer.md - typo (qmk#23275)
  [Keyboard] add glyphkbd_v2 (qmk#23131)
  Add support new kb_elmo PCBs (qmk#23138)
  ...
mzyt pushed a commit to mzyt/qmk_firmware that referenced this pull request May 6, 2024
girtsf pushed a commit to girtsf/qmk_firmware that referenced this pull request Jun 5, 2024
whoisjordangarcia pushed a commit to whoisjordangarcia/qmk_firmware that referenced this pull request Jun 8, 2024
nuess0r pushed a commit to nuess0r/qmk_firmware that referenced this pull request Sep 8, 2024
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.

3 participants