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

Format code #67

Closed
wants to merge 1 commit into from
Closed

Format code #67

wants to merge 1 commit into from

Conversation

aaronjheng
Copy link

No description provided.

@00JCIV00
Copy link
Owner

Hey, I appreciate your interest in Cova and that you took the time to contribute a PR.

In regards to Zig formatting, I'm not a fan of many of the changes it forces, so I specifically opt not to run zig fmt on the library.

Anyone who finds the formatted style easier to read can easily pull the code and format it locally of course. Also, your PR does bring to light the need to introduce contribution guidelines after the next version.

Hopefully that doesn't dissuade you from trying out the library.

@aaronjheng
Copy link
Author

In regards to Zig formatting, I'm not a fan of many of the changes it forces, so I specifically opt not to run zig fmt on the library.

Totally understand. Zig still has numerous aspects requiring further refinement.

@aaronjheng aaronjheng closed this Nov 16, 2024
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