Skip to content
This repository has been archived by the owner on Apr 16, 2020. It is now read-only.

-m alias for --type=module #39

Closed
wants to merge 12 commits into from

Conversation

GeoffreyBooth
Copy link
Member

Adds -m alias for --type=module. Was missing from #32 despite being added to the docs in that PR.

I’m new to the codebase so feedback very welcome.

Copy link
Contributor

@guybedford guybedford left a comment

Choose a reason for hiding this comment

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

Thanks for working on this.

guybedford and others added 11 commits February 24, 2019 09:06
Refs: nodejs/modules#180

PR-URL: nodejs#6
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Refs: nodejs/modules#180

PR-URL: nodejs#6
Reviewed-By: Guy Bedford <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Refs: nodejs/modules#180

PR-URL: nodejs#6
Reviewed-By: Guy Bedford <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Refs: nodejs/modules#180

PR-URL: nodejs#6
Reviewed-By: Guy Bedford <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
PR-URL: nodejs#6
Reviewed-By: Guy Bedford <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
PR-URL: nodejs#6
Reviewed-By: Guy Bedford <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
@Fishrock123
Copy link
Contributor

I'm not so into -m, but the other parts seem fine

@MylesBorins
Copy link
Contributor

Consensus in meeting that we can land

@GeoffreyBooth
Copy link
Member Author

Closing in favor of #45 which includes this same code and was also approved in the meeting.

@GeoffreyBooth GeoffreyBooth deleted the m-flag branch February 27, 2019 21:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants