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

Remove legacy architectures and flatten directory structure #57

Merged
merged 15 commits into from
Apr 23, 2024

Conversation

burkfers
Copy link
Collaborator

@burkfers burkfers commented Apr 8, 2024

As discussed.
Requires sync to https://github.com/0xcharly/bastardkb-qmk/tree/2024-04-07-bkb-master-sync first.

Todo:

  • charybdis/3x5
  • charybdis/3x6
  • charybdis/4x6
  • skeletyl
  • tbkmini
  • scylla
  • Update readmes to point to legacy tag/hash

Discuss:

  • Remove dilemma/3x5_2?
    • flattened into assembled/ as the only variant
  • Remove tbk?
    • removed
  • Do DD conversion in one fell swoop?

@github-actions github-actions bot removed documentation Improvements or additions to documentation core via CI translation dd labels Apr 8, 2024
@burkfers burkfers marked this pull request as ready for review April 8, 2024 14:34
@burkfers
Copy link
Collaborator Author

burkfers commented Apr 8, 2024

"ready for review" was hasty, apologies.
dilemma 3x5_2 readme and final checks for references to legacy architectures pending.

@burkfers burkfers marked this pull request as draft April 8, 2024 14:40
@burkfers burkfers marked this pull request as ready for review April 9, 2024 09:06
@burkfers burkfers requested a review from 0xcharly April 22, 2024 10:20
@burkfers burkfers force-pushed the bkb-master-sync-flattened branch from e71163a to 3db7498 Compare April 22, 2024 10:32
@0xcharly 0xcharly merged commit 70e703a into Bastardkb:bkb-master Apr 23, 2024
2 checks passed
@0xcharly
Copy link
Collaborator

Amazing work, thanks a ton, @finrod09 !

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.

2 participants