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

Ghost Type Encodings #181

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Conversation

aakp10
Copy link

@aakp10 aakp10 commented Sep 23, 2020

This PR adds encoding definitions for Ghost Types.

Copy link
Contributor

@vakaras vakaras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me so far. Would be good now to add some tests that illustrate how it currently works.

@fpoli
Copy link
Member

fpoli commented Sep 28, 2020

I totally agree that we should have some tests before merging.

@aakp10
Copy link
Author

aakp10 commented Sep 28, 2020

I totally agree that we should have some tests before merging.

@vakaras @fpoli Thanks for the reviews, I've added a couple of type checking related test cases for the ghost types. I'll continue extending the test cases to accommodate the noninterference properties of ghost types with the upcoming changes that I'm currently working on.

@aakp10 aakp10 force-pushed the karuna/encoding-v2 branch 3 times, most recently from 5db62e1 to f5de7f8 Compare October 5, 2020 17:47
@aakp10 aakp10 force-pushed the karuna/encoding-v2 branch 2 times, most recently from cd6e20a to 3beeb22 Compare October 10, 2020 07:52
@vakaras vakaras assigned vakaras and unassigned aakp10 Nov 2, 2020
@aakp10 aakp10 force-pushed the karuna/encoding-v2 branch 2 times, most recently from 7976ee3 to a27f3fd Compare November 25, 2020 07:44
@aakp10 aakp10 force-pushed the karuna/encoding-v2 branch from a27f3fd to 3895c62 Compare December 21, 2020 13:11
@Aurel300 Aurel300 assigned Aurel300 and unassigned vakaras Mar 8, 2021
@Aurel300 Aurel300 force-pushed the karuna/encoding-v2 branch from acf1667 to 7ea498d Compare March 10, 2021 18:22
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.

4 participants