Skip to content

Error when importing shadowed internal module. #264

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 28, 2014
Merged

Error when importing shadowed internal module. #264

merged 1 commit into from
Jul 28, 2014

Conversation

ahejlsberg
Copy link
Member

Fixes #73.

@DanielRosenwasser
Copy link
Member

Looks good, although, it's not obvious why you need the first identifier until you look at the spec.

In any case, 👍.

@vladima
Copy link
Contributor

vladima commented Jul 27, 2014

👍

ahejlsberg added a commit that referenced this pull request Jul 28, 2014
@ahejlsberg
Merge pull request #264 from Microsoft/errorOnHiddenModuleImport
fc00047 
Error when importing shadowed internal module.
@ahejlsberg ahejlsberg merged commit fc00047 into master Jul 28, 2014
@ahejlsberg ahejlsberg deleted the errorOnHiddenModuleImport branch July 28, 2014 01:09
@microsoft microsoft locked and limited conversation to collaborators Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Errors] No error trying to import shadowed internal module
3 participants
@ahejlsberg @DanielRosenwasser @vladima