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

Added Asset, Colemak, Colemak-DH, Niro, Soul, and Workman layouts #71

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

NaeemBolchhi
Copy link

What is it?

  • Bugfix
  • Feature
  • Codebase improvement

Description of the changes in your PR

Screenshots

  • Asset:
  • Colemak:
  • Colemak-DH:
  • Niro:
  • Soul:
  • Workman:

Acknowledgement

@Aga-C
Copy link
Member

Aga-C commented Aug 18, 2024

Thanks for contributing, but next time please create an issue or discussion first (as stated in the contribution guidelines).

@NaeemBolchhi
Copy link
Author

NaeemBolchhi commented Aug 18, 2024

Apologies about not creating an issue first. It wasn't clear to me that I needed to do it (maybe make it a bit clearer?). It says to make sure that the new feature or bug is already reported and is open to pull requests, but does not mention that I should open an issue myself if one does not already exist. I'm not a native English speaker, so I apologize again if this was obvious and I misunderstood. Should I open an issue now?

@Aga-C
Copy link
Member

Aga-C commented Aug 18, 2024

No need to open a new issue, just remember for the future 🙂

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