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

Merge master into release #601

Merged

Conversation

DustinCampbell
Copy link
Member

No description provided.

DustinCampbell and others added 30 commits June 30, 2016 12:43
This fixes debugger restoration on osx.10.12.
Move OmniSharp-specific code into omnisharp directory
Update vscode and vsce module versions and vscode engine version to latest
Hardcode osx.10.11-x64 for osx runtime id
These package updates turn on conditional breakpoint support at the UI
level and step filtering support.
Update MIEngine reference and add step filtering to launch schema
* Update link to .NET Core tools

* Fix link
Enabling action pickProcess for attach.
Red Hat Enterprise Linux.
Fixed typo. This is a blocker bug for RHEL users because this extension can't launch.
Update version to 1.3.0 for master branch
In a code path, template processId is 0 instead of command.pickProcess. Fixing it.
This gives us more control over which RID to use for linux distros.
Previously we were manually parsing the path environment variable which is
prone to issue when users $PATH variables are malformed. This should be
more robust.
check if dotnet is on $PATH by executing 'dotnet --info'
Determine linux RID's based on etc/os-release rather than dotnet --info
DustinCampbell and others added 21 commits July 20, 2016 07:29
It is no longer necessary to ensure that the workspace root contains a project.json (or global.json) before offering
to generate tasks.json and launch.json. When launched, OmniSharp should scan for projects and, if any or found, we'll
offer to generate the assets if needed.
1. The diagnostics provider won't refresh diagnostics while the OmniSharp server is restoring packages. Instead, it
   waits until it receives an event from OmniSharp afterward. However, that event was only expecting the old DNX
   protocol, so the diagnostics provider never automatically updated after packages were restored. This change gets
   rid of the DNX protocol and changes the diagnostic provider to look for .NET Core projects.

2. Hidden diagnostics should have DiagnosticSeverity.Info rather DiagnosticSeverity.Warning.

3. When the processing *all* of the diagnostics (not just those for the current file), the previous diagnostics were
   not cleared out first. This resulted in new diagnostics being merged with old diagnostics, creating duplicates
   diagnostics. Now we clear the diagnostics for a file before adding new ones.
Improve generation of tasks.json and launch.json assets
…inux-distro-text

Tweak the message displayed when trying to acquire OmniSharp on an unsupported Linux distro
Update coreclr-debug packages for 1.3 release.
…ssue

Fix relative path issue in asset generation
…-install

Fix offline packaging for Windows to include both the CoreCLR and Desktop versions of OmniSharp
@DustinCampbell DustinCampbell added this to the 1.3 milestone Jul 20, 2016
@DustinCampbell
Copy link
Member Author

@gregg-miskelly, @chuckries, @caslan, @rajkumar42: I scanned through the changes and it looks right to me. Do any of you want to double-check the debugger changes?

@gregg-miskelly
Copy link
Contributor

@DustinCampbell Yeah, we were going to do a quick smoke test of the .vsix. Do you have one built already? (If not, we can build it)

@DustinCampbell
Copy link
Member Author

We should build the final from the release branch. Once I merge, could you build it from there?

@gregg-miskelly
Copy link
Contributor

That works with me.

@DustinCampbell
Copy link
Member Author

look good?

@gregg-miskelly
Copy link
Contributor

Yup. Merge it.

@DustinCampbell DustinCampbell merged commit a2e2142 into dotnet:release Jul 21, 2016
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.

9 participants