Skip to content

Conversation

@alabs-tomscholz
Copy link

No description provided.

@henhal
Copy link
Owner

henhal commented Oct 16, 2025

Hi @alabs-tomscholz

I really appreciate the PR but haven't had time to review yet. Did you close based on lack of response, in that case I apologize. I do intend to review this as soon as possible.

@alabs-tomscholz
Copy link
Author

Hi Henrik,

we're using your project for some of our projects and wanted to upgrade the dependencies. While doing that we also started to do some more refactoring. We've published a new version on npm that works for us right now, but if you would like, we could also merge those changes back into your project, but then some changes would need to be done, like updating the README and fixing one or two bugs that got introduced by our refactoring.

Are you still using this package yourself?

Kind regards from Germany,
Tom

… types error

like Error: unknown format "date-time" ignored in schema at path
@henhal
Copy link
Owner

henhal commented Oct 22, 2025

Hi @alabs-tomscholz!
Happy to hear you've find this package useful. Yes, I'm using it myself and I would be interested in reviewing your changes with the intent to merge them.

@alabs-tomscholz
Copy link
Author

Alrighty. Just fyi that we changed all the import to our new package, that would have to get changed, might make sense that you branch off from this updates branch and update the paths and imports there :)

@luqasn
Copy link
Contributor

luqasn commented Oct 22, 2025

I have rebased our changes to remove the ownership change and clean it up a little for your reviewing pleasure :)
#10 is the new one, this can be closed.

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.

3 participants