Skip to content

Conversation

@s3cur3
Copy link
Contributor

@s3cur3 s3cur3 commented Nov 24, 2025

Hi David! 👋 Thanks for this! I'm using it when parsing dates in WHOIS records for the Whois package.

This PR adds support for dates formatted like:

  • Sunday 5th Jan 2025
  • Tuesday 2nd Feburary 2024
  • 1st January 2025

This adds support for dates formatted like:

- Sunday 5th Jan 2025
- Tuesday 2nd Feburary 2024
- 1st January 2025
@s3cur3 s3cur3 force-pushed the weekday-day-month-year branch from ff57067 to f5ed29e Compare November 24, 2025 14:27
@s3cur3
Copy link
Contributor Author

s3cur3 commented Nov 24, 2025

Oh no, the tests are failing! I swear they passed locally... I'll look into it...

@s3cur3
Copy link
Contributor Author

s3cur3 commented Nov 24, 2025

Oh! I think the tests are failing because I didn't commit the modified lib/combinators.ex. (I see the same set of 30 errors if I reset my working copy to what's in the commit.) It looks like CI doesn't run mix compile.nimble?

@dbernheisel
Copy link
Owner

@s3cur3 correct, I can get to it or you can, but otherwise LGTM.

@s3cur3
Copy link
Contributor Author

s3cur3 commented Nov 24, 2025

Ah, very good! I've added it. (Would you like me to also edit the contributing guide to remove the instruction not to commit the compiled NimbleParsec artifacts?)

@dbernheisel
Copy link
Owner

@s3cur3 I remember why I did it this way -- I wanted to review the exs file and not the ex (it gets so large). I don't want contributors to trip over this next time, so I'll adjust the CI process to generate the ex file before running tests, and the publishing process will ensure it's generated and up to date.

I'll take care of this tonight! Thanks for the contribution ❤️

@dbernheisel dbernheisel merged commit 960be02 into dbernheisel:main Nov 25, 2025
3 checks passed
@s3cur3
Copy link
Contributor Author

s3cur3 commented Nov 25, 2025

Ah, understood. Thank you!

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