-
-
Notifications
You must be signed in to change notification settings - Fork 556
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
Improve tokei's language test coverage. #63
Comments
I could start working on some languages on the weekends if it is okay for the terms |
@Emulebest Contribute when you can. All contributions are welcome. |
I'd also like to help with some language tests as part of Hacktoberfest this year. |
@JHaig343 Sure! |
@XAMPPRocky I noticed that proper execution of the test file requires the top comment to be in the file and in the correct format(X lines X code X comments X blanks). The contributing doc for Tests does not mention that the format is required. Would I be able to make a PR to explicitly state the required format in CONTRIBUTING.md? |
There should be a file for each language that covers every edge case. The file should contain every variant comments and quotes, as well as a comment at the top of the file containing the manually verified lines, code, comments, blanks e.g.
// 39 lines 32 code 2 comments 5 blanks
. A good example of a test file istests/data/rust.rs
.Languages
The text was updated successfully, but these errors were encountered: