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

fix: --no-check not properly handling code nested in TS expressions #12416

Merged
merged 2 commits into from Oct 13, 2021

Conversation

dsherret
Copy link
Member

The issue here was that a resolver_with_mark was used since that's what the documentation said to use, but because we're using TypeScript we should have used a ts_resolver instead.

Closes #12409, #12413

@dsherret dsherret requested a review from kitsonk October 13, 2021 00:32
Copy link
Contributor

@kitsonk kitsonk left a comment

Choose a reason for hiding this comment

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

LGTM

@dsherret dsherret merged commit 51f680f into denoland:main Oct 13, 2021
@dsherret dsherret deleted the fix_no_check_renaming branch October 13, 2021 01:27
@panva
Copy link
Contributor

panva commented Oct 13, 2021

Thank you for a quick resolution. Is this a candidate for a quick 1.15.1?

@lucacasonato
Copy link
Member

@panva 1.15.1 coming up in about 50 minutes.

@lambdalisue
Copy link

lambdalisue commented Oct 13, 2021

It seems Deno v1.15.1 is not released yet (5 hours passed from previous comment).

It seems CI fails and stack?
https://github.com/denoland/deno/actions/runs/1336487877

@lucacasonato
Copy link
Member

@lambdalisue Yeah GitHub Actions is having some issues. We are about 95% there now. Give us a couple more minutes.

@lambdalisue
Copy link

lambdalisue commented Oct 13, 2021

Understand. Thanks for Deno teams great work 🥇

@lambdalisue
Copy link

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.

--no-check badly renaming symbols
5 participants