-
Notifications
You must be signed in to change notification settings - Fork 77
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: adds a brief helpful comment to each entry describing its function #593
feat: adds a brief helpful comment to each entry describing its function #593
Conversation
The name 'affixes' turns out to be a misnomer since about a quarter or a third of them don't add any prefix or suffix but change a property such as the part-of-speech, or singular vs plural. Another approach would be to switch from `json` to either [`JSON5`](https://json5.org/) or [`Hjson`](https://hjson.github.io/)
Here's a bonus nodejs/bun oneliner to make a suffix cheat sheet! node -e "Object.entries(require('./harper-core/affixes.json').affixes).forEach(([key, value]) => { if (value['#']) console.log(key + ': ' + value['#']); });" Will output:
|
Love it. Why not add that as a |
Slight pain finding `{{justfile_directory()}}` needed for this transition. Took the opportunity to write the js in a clearer, hopefully self-documenting way
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thanks for helping build out this documentation
A pleasure! |
The name 'affixes' turns out to be a misnomer since about a quarter or a third of them don't add any prefix or suffix but change a property such as the part-of-speech, or singular vs plural.
Another approach would be to switch from
json
to eitherJSON5
orHjson
for theaffixes.json
file.