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

BREAKING(cli/eval): remove --ts flag #12198

Closed
wants to merge 2 commits into from

Conversation

satyarohith
Copy link
Member

@satyarohith satyarohith commented Sep 23, 2021

Use deno eval --ext instead.

Should be merged after the merge window for 2.0 opens (27 October).

@satyarohith satyarohith added this to the 2.0.0 milestone Sep 23, 2021
@kitsonk kitsonk added this to Done in Deno CLI 2.0 Sep 23, 2021
@kitsonk
Copy link
Contributor

kitsonk commented Sep 23, 2021

Is there a related issue?

@satyarohith
Copy link
Member Author

Is there a related issue?

No. The PR addresses a TODO that was added when we introduced the --ext flag.

@kitsonk
Copy link
Contributor

kitsonk commented Sep 25, 2021

I think even with minor scope, we should probably discuss its inclusion. Especially TODOs in codes might have seen like a good idea at some point, but we should discuss it for inclusion as well as track it as part of the 2.0 scope. I have added it to the project board and will add it to the release plan.

@kitsonk kitsonk mentioned this pull request Sep 25, 2021
17 tasks
@CLAassistant
Copy link

CLAassistant commented Oct 15, 2021

CLA assistant check
All committers have signed the CLA.

@bartlomieju
Copy link
Member

2.0 was postponed indefinitely, so let's close this PR for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants