Using Zod as validator might make things more complex to maintain #47

New issue

Closed

opened 2025-04-24 23:14:36 +00:00 by lucmsilva651 · 3 comments

lucmsilva651 commented

2025-04-24 23:14:36 +00:00

(Migrated from github.com)

@lucmsilva651 What do you think about this? Using Zod might make things more complex for you to maintain in terms of modifying types.

Originally posted by @ihatenodejs in https://github.com/abocn/TelegramBot/pull/46#discussion_r2059361824

> @lucmsilva651 What do you think about this? Using [Zod](https://zod.dev) might make things more complex for you to maintain in terms of modifying types. _Originally posted by @ihatenodejs in https://github.com/abocn/TelegramBot/pull/46#discussion_r2059361824_

ihatenodejs commented

2025-04-24 23:20:04 +00:00

(Migrated from github.com)

To provide a longer explanation:

I don't want to make it harder for you to contribute to Kowalski because of this migration, especially if we use Zod.

I've seen that Zod is very helpful, and I have used it in some of my projects. However, this is another thing you have to learn, so I do not want to make too many changes

To provide a longer explanation: I don't want to make it harder for you to contribute to Kowalski because of this migration, especially if we use Zod. I've seen that Zod is very helpful, and I have used it in some of my projects. However, this is another thing you have to learn, so I do not want to make too many changes

GiovaniFZ commented

2025-04-24 23:45:22 +00:00

(Migrated from github.com)

@lucmsilva651 basically, if we use Zod, code quality will be better but it's up to you if you want to use or not, so what's your opinion about? That's what we want to know

lucmsilva651 commented

2025-04-25 14:28:59 +00:00

(Migrated from github.com)

@GiovaniFZ @ihatenodejs I will merge this PR soon, so please put Zod support into it.

Note: I don't really know how to use Zod, but since any JavaScript code can be valid TypeScript code, you guys can do the job of porting my new code to TypeScript on future.

@GiovaniFZ @ihatenodejs I will merge this PR soon, so please put Zod support into it. Note: I don't really know how to use Zod, but since any JavaScript code can be valid TypeScript code, you guys can do the job of porting my new code to TypeScript on future.

👍 1