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

Putting all IBM Model M projects under one folder #12370 #12792

Closed
wants to merge 2,145 commits into from

Conversation

nuess0r
Copy link
Contributor

@nuess0r nuess0r commented May 3, 2021

As discussed in #12370:
The maintainers of several Model M replacement
controller projects agreed to cosolidate them
under one common folder.

Description

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 keyboard keymap via Adds via keymap and/or updates keyboard for via support labels May 3, 2021
@zvecr zvecr changed the base branch from master to develop May 4, 2021 00:23
@zvecr zvecr added the breaking_change Changes that need to wait for a version increment label May 4, 2021
@drashna drashna requested a review from a team May 27, 2021 03:47
@nuess0r
Copy link
Contributor Author

nuess0r commented Jul 15, 2021

any news on this pull request?
Seems that none of the Model M developers has a need to comment here or a wish to change something.

@drashna
Copy link
Member

drashna commented Jul 20, 2021

Looks like you have a merge conflict that needs to be resolved, at the very least.

@nuess0r
Copy link
Contributor Author

nuess0r commented Jul 20, 2021

Merge conflict solved

@drashna
Copy link
Member

drashna commented Jul 20, 2021

It looks like you may need to rebase to properly fix this.

@nuess0r
Copy link
Contributor Author

nuess0r commented Jul 20, 2021

OK, I tried to follow your guide: https://docs.qmk.fm/#/newbs_git_resolving_merge_conflicts

This is a new process for me. Hope I did it right.

@drashna
Copy link
Member

drashna commented Jul 30, 2021

it looks like you merged, rather than rebased. Unfortunately that leaves it in the same position.

If you want, I should be able to see about helping out with this.

Pylon and others added 11 commits September 19, 2021 11:47
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
revert lib/chibios

revert lib/chibios

Finish up

remove files

Update copyrights

remove copyrights

readd copyrights

add stop macro

PR suggestions

PR suggestions

readme lowercase

fix rotate for oled

Co-authored-by: Z003YW4 <ben.lipson@target.com>
precondition and others added 24 commits October 20, 2021 01:26
* reformat info.json

- friendly-format info.json
- move the right half to the left by 0.5u (improves readability, for me anyway)
- correct key sequence

* refactor default keymap

- use short-form keycodes
- use four-space indent

* refactor via keymap

- use short-form keycodes
- use four-space indent

* update maintainer field in info.json

Use the maintainer's GitHub username.

* readme touch-up

Minor edit to the lead paragraph, and reformat the Hardware Availability section.

* change BOOTMAGIC_ENABLE from `lite` to `yes`
Co-authored-by: Ryan <fauxpark@gmail.com>
…sistent (qmk#14902)

Co-authored-by: Ingo Junefjord <ingo.junefjord@attentec.se>
As discussed in qmk#12370:
The maintainers of several Model M replacement
controller projects agreed to cosolidate them
under one common folder.
@github-actions github-actions bot removed the cli qmk cli command label Oct 26, 2021
@nuess0r
Copy link
Contributor Author

nuess0r commented Oct 26, 2021

Today I invested some other hour to rebase my changes.

I even found a new project for the Model M SSK that appeared in the meantime. I moved that as well.

Github shows now still two merge conficts, but I don't understand why. Is it possible to merge it like that or do you need something more from me?

@drashna
Copy link
Member

drashna commented Oct 27, 2021

It looks like it needs a rebase, rather than a merge.

@nuess0r nuess0r mentioned this pull request Oct 31, 2021
14 tasks
@nuess0r
Copy link
Contributor Author

nuess0r commented Oct 31, 2021

I did try to rebase but seems that long ago I did a merge by mistake. I'm new to rebase, never needed that at work.
I didn't knew how to fix it in this branch so I created a new branch and cherry-picked what was correct: #14996

So I close this one.

@nuess0r nuess0r closed this Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking_change Changes that need to wait for a version increment CI core documentation keyboard keymap python translation via Adds via keymap and/or updates keyboard for via support
Projects
None yet
Development

Successfully merging this pull request may close these issues.