Skip to content

Abolish ‘yaml’ configuration#157

Merged
mrkkrp merged 2 commits intomasterfrom
mk/abolish-yaml-config
Jul 7, 2019
Merged

Abolish ‘yaml’ configuration#157
mrkkrp merged 2 commits intomasterfrom
mk/abolish-yaml-config

Conversation

@mrkkrp
Copy link
Copy Markdown
Member

@mrkkrp mrkkrp commented Jul 7, 2019

What could be better?

@mrkkrp mrkkrp force-pushed the mk/abolish-yaml-config branch from bd9354f to 5a1e291 Compare July 7, 2019 11:41
@mrkkrp
Copy link
Copy Markdown
Member Author

mrkkrp commented Jul 7, 2019

I'm not 100% sure about the set of extensions, but it's a start.

@mrkkrp mrkkrp merged commit 01486ed into master Jul 7, 2019
@mrkkrp mrkkrp deleted the mk/abolish-yaml-config branch July 7, 2019 11:54
@eschnett
Copy link
Copy Markdown

eschnett commented Jul 7, 2019

Yay, thanks!

Out of curiosity: Is that a set of extensions that is used somewhere in production? Is that e.g. a default set of extensions used at tweag.io?

@mboes
Copy link
Copy Markdown
Contributor

mboes commented Jul 7, 2019

No, it's a conservative set of "non-keyword stealing" language extensions. But I think we should not have merged such an explicit list, which will have to be reviewed at each new compiler version, because that's unnecessary busywork as argued in #62 (comment).

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

Labels

None yet

Projects

None yet

3 participants