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

fix(compiler): add a nested property to the test cases #140

Closed
wants to merge 2 commits into from
Closed

fix(compiler): add a nested property to the test cases #140

wants to merge 2 commits into from

Conversation

phil-lgr
Copy link

See issue #139

@CLAassistant
Copy link

CLAassistant commented Jul 27, 2020

CLA assistant check
All committers have signed the CLA.

@roperzh
Copy link
Member

roperzh commented Aug 4, 2020

@phil-lgr thank you very much for this! are you interested in working on a fix for this test case? I can help you if you are, but I can also take this if you aren't.

@phil-lgr
Copy link
Author

phil-lgr commented Aug 4, 2020

hey @roperzh I would really appreciate if you could have look yes, the only changes worth taking are on this Primitives.ts file: https://github.com/diez/diez/blob/e4f9a4ff0961afce1265dc69f5d982a00135b8df/src/compiler/targets/test/fixtures/Primitives/Primitives.ts

I guess you could copy paste this to start?

@phil-lgr phil-lgr closed this Dec 16, 2020
@phil-lgr phil-lgr deleted the fix-nested-properties branch December 16, 2020 10:11
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.

None yet

3 participants