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

Address #305: function names in error messages #306

Merged
merged 1 commit into from
Aug 10, 2024

Conversation

rmolinari
Copy link
Contributor

Some functions are named incorrectly in their own error messages.

Examples:

  • split\every appears as "split-every" in its own error messages
    • most of the cases I found have this form: backslash appears as hyphen
  • to-eyr and eyr\full both appear as "eyr"
    • this appears to be a copy/paste issue; there are some other occurrences
  • near-zero appears as "is-zero"
    • perhaps an artifact of a renaming
    • this function's descriptive string was also wrong

The commit contains the cases that I found. I don't have a systematic way to check all the error messages, so there are probably still some more cases.

Some functions are named incorrectly in their own error messages.

Examples:
- split\every appears as "split-every" in its own error messages
  - most of the cases I found have this form: backslash appears as
    hyphen
- to-eyr and eyr\full both appear as "eyr"
  - this appears to be a copy/paste issue; there are some other
    occurrences
- near-zero appears as "is-zero"
  - perhaps an artifact of a renaming
  - this function's descriptive string was also wrong
Copy link
Owner

@refaktor refaktor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you!

@refaktor refaktor merged commit 3b92ce3 into refaktor:main Aug 10, 2024
8 checks passed
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