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

Prototype #references, #files, and #imports #7077

Closed
wants to merge 3 commits into from

Conversation

amcasey
Copy link
Member

@amcasey amcasey commented Nov 25, 2015

  • #references - referenced assemblies
  • #files - loaded files
  • #imports - imported namespaces/types/aliases

They should work in both csi and the C# interactive window. All but #imports should also work in VB, to the extent that VB works at all.

Caveat: #references will not show references inherited from previous submissions until #6587 is fixed.

`#references` - referenced assemblies
`#files` - loaded files
`#imports` - imported namespaces/types/aliases

They should work in both csi and the C# interactive window.  All but
`#imports` should also work in VB, to the extent that VB works at all.

Caveat: `#references` will not show references inherited from previous
submissions until dotnet#6587 is fixed.
@amcasey
Copy link
Member Author

amcasey commented Nov 25, 2015

FYI @dotnet/roslyn-interactive please give this a whirl and let me know what you think of it. If the behavior seems desirable, I'll lock it in with tests.

@dpoeschl
Copy link
Contributor

retest prtest/win/dbg/unit32 please
// Previous failure: http://dotnet-ci.cloudapp.net/job/roslyn_prtest_win_dbg_unit32/1604/
// Retest reason: Testing StashPop (sorry for the noise Andrew)

@heejaechang
Copy link
Contributor

@amcasey I believe this is specific to your PR?

@amcasey
Copy link
Member Author

amcasey commented Dec 2, 2015

@heejaechang yes, will fix.

@amcasey
Copy link
Member Author

amcasey commented Jan 6, 2016

@tannergooding Why don't you take over the branch, since it dovetails so nicely with your proposal? You might want to build it locally and play around with the commands to get a feel for this design - it's totally experimental and open to improvements.

Things to consider:

  1. There are no tests of this code. I didn't want to lock in experimental behavior.
  2. #references doesn't include references from previous submissions. This code calls the correct API, but the API is broken. See @tmat for details.

@amcasey amcasey closed this Jan 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants