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

Strict #104

Merged
merged 5 commits into from
Sep 16, 2014
Merged

Strict #104

merged 5 commits into from
Sep 16, 2014

Conversation

forki
Copy link
Member

@forki forki commented Sep 15, 2014

Implements the references strict mode as discussed in #38

/cc @agross @ilkerde

@agross
Copy link
Contributor

agross commented Sep 15, 2014

Looks good sample code-wise. Haven't tested yet. I believe there some extra indent in the docs code section, but perhaps caused by my phone.

@forki
Copy link
Member Author

forki commented Sep 15, 2014

@ilkerde @agross please review the test cases for the dependencies and lock files. The other stuff can be fixed, but we should get the syntax right.

@forki forki merged commit 4e77a3f into master Sep 16, 2014
@bartelink
Copy link
Member

@forki Did you consider #38 (comment) // @agross @ilkerde

I really don't think modal stuff works well in what should be a declarative language (related: #95 )

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