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.

Sign up for GitHub

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

Jump to bottom

Fix compilation error when setting DateTime.t() as a field type #76

Closed

florius0 wants to merge 0 commits into witchcrafters:main from florius0:main

florius0 commented Jun 18, 2022

This PR fixes #75

After a brief investigation it appears that Algae treats any Module.t() (except for string) as an Algae type, which all define new/0.

So my proposal is to check if module defines new/0 and if it does not, use nil instead

florius0 changed the title ~~Fix witchcrafters/algae#75~~ Fix compilation error when setting DateTime.t() as a field type

florius0 closed this

florius0 force-pushed the main branch from a2f8ed0 to 8e9e248 Compare

June 22, 2022 18:11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment