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

Streamlined Version #65

Merged
merged 1 commit into from
May 24, 2024
Merged

Streamlined Version #65

merged 1 commit into from
May 24, 2024

Conversation

MonkeyIsNull
Copy link
Contributor

@MonkeyIsNull MonkeyIsNull commented May 23, 2024

This is a severely minimalistic version. I've stripped it down to nothing but the essentials. That said it has no flexibility, no clear debugging object, and if you don't put a postPath and don't declare in the method that it is a partialValue or a emptyValue, then no soup for you.

@MonkeyIsNull
Copy link
Contributor Author

MonkeyIsNull commented May 24, 2024

There may be a middle ground somewhere between this version and the last pull request, but when I coded it up I hated it. Somehow it seemed worse to me, so I got frustrated and tried to see if I could do something that just went straight at it and nothing else.

@anewton1998 anewton1998 merged commit b625d74 into icann:dev May 24, 2024
2 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