Skip to content

More exception throwing and handling#3

Open
datvo06 wants to merge 4 commits intomainfrom
http_edgecase_handling
Open

More exception throwing and handling#3
datvo06 wants to merge 4 commits intomainfrom
http_edgecase_handling

Conversation

@datvo06
Copy link
Copy Markdown
Contributor

@datvo06 datvo06 commented Mar 3, 2025

No description provided.

@datvo06 datvo06 requested a review from zenna March 3, 2025 19:43
@datvo06 datvo06 marked this pull request as draft March 3, 2025 19:45
@datvo06 datvo06 self-assigned this Mar 3, 2025
@datvo06
Copy link
Copy Markdown
Contributor Author

datvo06 commented Mar 3, 2025

It's still under development, I put it here for easier debugging.

@datvo06 datvo06 marked this pull request as ready for review March 3, 2025 21:14
@datvo06
Copy link
Copy Markdown
Contributor Author

datvo06 commented Mar 3, 2025

I fixed the error checking. Previously, if "t isa Maybe" and f not in the json key, it adds nothing. It's not correct since t is a type, this is always true.
I'm fixing it to use Julia dispatcher to throw errors.

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.

1 participant