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

feat: create command helpers #6

Merged
merged 5 commits into from
Jan 5, 2022
Merged

Conversation

karitham
Copy link
Owner

@karitham karitham commented Jan 4, 2022

also move _example to 0_example since gopls doesn't like the _example directory name.

Karitham added 2 commits January 4, 2022 18:25
also move _example to 0_example since gopls
doesn't like the _example directory name.
@jolheiser
Copy link
Collaborator

Meh, screwed up my branch. Check out 6d69693

@jolheiser
Copy link
Collaborator

As mentioned in discord, I think generation might be best for this, if at all.

Generics attempt was pain.

@karitham
Copy link
Owner Author

karitham commented Jan 5, 2022

Seems like the case, yeah,

My efforts are currently focused around #7, but will come back to that once I'm done.

Open to any other suggestions, truthfully, the generics part was a good idea from a user POV, but it's an ugly and non-extensible hack.

@karitham karitham requested a review from jolheiser January 5, 2022 20:11
@jolheiser jolheiser merged commit b03180c into master Jan 5, 2022
@karitham karitham deleted the feat/createCommandHelpers branch January 6, 2022 08:38
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