Skip to content

Conversation

weswigham
Copy link
Member

We can pass empty lists via tsconfig.json, so this just helps to lessen the disparity a bit. You get an empty list when a list-taking command is followed by another command or a single comma.

@weswigham
Copy link
Member Author

weswigham commented Jun 10, 2016

@yuit @mhegazy I think you both originally implemented list compiler options? So you should probably review this.

start: undefined,
length: undefined,
}, {
messageText: "Argument for '--lib' option must be: 'es5', 'es6', 'es2015', 'es7', 'es2016', 'es2017', 'dom', 'webworker', 'scripthost', 'es2015.core', 'es2015.collection', 'es2015.generator', 'es2015.iterable', 'es2015.promise', 'es2015.proxy', 'es2015.reflect', 'es2015.symbol', 'es2015.symbol.wellknown', 'es2016.array.include', 'es2017.object', 'es2017.sharedmemory'",
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't all 3 cases be errors? I don't think lib: [] is valid, so somebody along the line should throw an error.

Copy link
Member

Choose a reason for hiding this comment

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

I learned in offline discussion with @weswigham that lib: [] IS valid, so carry on! :)

@mhegazy
Copy link
Contributor

mhegazy commented Jun 10, 2016

👍

@sandersn
Copy link
Member

👍 and thanks for creating the issue to add git-style "-- is the end-of-options option"

@DanielRosenwasser
Copy link
Member

What about requests for - to mean stdin/stdout?

@weswigham
Copy link
Member Author

Do we do that already? If so the simple fix is probably just to scan two characters (looking for any nonwhitespace character as the second) from the next argument string rather than just one. If we don't I could add such a check in preparation.

@mhegazy
Copy link
Contributor

mhegazy commented Jun 13, 2016

What about requests for - to mean stdin/stdout?

we decided it was too much majic. i would rather have --writeToSTDOut instead.

@DanielRosenwasser
Copy link
Member

:shipit: ❗ ❗ ❗ ❗ ❗

@weswigham weswigham merged commit 9d023a8 into microsoft:master Jun 14, 2016
@weswigham weswigham deleted the allow-empty-lists-command-line branch October 12, 2017 08:05
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants