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

Increase metadata coverage for speced/fn #65

Open
vemv opened this issue Jun 6, 2019 · 0 comments
Open

Increase metadata coverage for speced/fn #65

vemv opened this issue Jun 6, 2019 · 0 comments

Comments

@vemv
Copy link
Contributor

vemv commented Jun 6, 2019

Problem statement

speced/fn lacks specific coverage for metadata (type hint) emission. Its coverage is only concerned with :pre/:post emission.

Note that the implementation is shared with that of defn, which is fully covered, so it's nearly impossible that the shared code does not work for the fn case.

The difficulty lies in that one cannot access fn metadata.

Proposal

Inspect the speced/fn macroexpansions, verifying its contained metadata.

Alternatives and comparison

  • Do nothing
    • As mentioned, it can be acceptable, but I'd rather keep things proven.
@vemv vemv mentioned this issue Jun 6, 2019
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant