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

Add ignore button to the MacInterpreterSelectedAndNoOtherInterpreters… #4808

Merged
merged 3 commits into from
Mar 20, 2019
Merged

Add ignore button to the MacInterpreterSelectedAndNoOtherInterpreters… #4808

merged 3 commits into from
Mar 20, 2019

Conversation

alinalobast
Copy link

@alinalobast alinalobast commented Mar 18, 2019

Add ignore button to the MacInterpreterSelectedAndNoOtherInterpretersDiagnostic diagnostic message to provide the ability to opt out of the warning. (Fixing related issue #4448)

For #4448

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Has a news entry file (remember to thank yourself!)
  • Has sufficient logging.
  • Has telemetry for enhancements.
  • Unit tests & system/integration tests are added/updated
  • Test plan is updated as appropriate
  • package-lock.json has been regenerated by running npm install (if dependencies have changed)
  • The wiki is updated with any design decisions/details.

@alinalobast
Copy link
Author

@brettcannon

…Diagnostic diagnostic message to provide the ability to opt out of the warning. (Fixing related issue #4448)
@codecov
Copy link

codecov bot commented Mar 18, 2019

Codecov Report

Merging #4808 into master will decrease coverage by 15%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #4808     +/-   ##
========================================
- Coverage      77%     62%    -14%     
========================================
  Files         450     373     -77     
  Lines       21707   14670   -7037     
  Branches     3573    1163   -2410     
========================================
- Hits        16673    9074   -7599     
- Misses       5030    5394    +364     
- Partials        4     202    +198
Flag Coverage Δ
#Linux ?
#Windows ?
#macOS ?

@alinalobast
Copy link
Author

Also, forgot to ask, looks like there are 2 diagnostic messages, and I was adding an Ignore button only to one of them as per the issue. But what about the other message, MacInterpreterSelectedAndHaveOtherInterpretersDiagnostic, should it have an Ignore button as well for consistency?

@DonJayamanne
Copy link

I'll add it.

@DonJayamanne DonJayamanne merged commit c014788 into microsoft:master Mar 20, 2019
DonJayamanne pushed a commit that referenced this pull request Mar 20, 2019
#4808)

Add ignore button to the MacInterpreterSelectedAndNoOtherInterpretersDiagnostic & MacInterpreterSelectedAndHaveOtherInterpretersDiagnostic diagnostic message to provide the ability to opt out of the warning. (Fixing related issue #4448)
DonJayamanne added a commit that referenced this pull request Mar 29, 2019
* Update tpn distro and 3rd party notices

* Pin to beta version of PTVSD (#4836)

Update version of PTVSD

* Add ignore button to the MacInterpreterSelectedAndNoOtherInterpreters… (#4808)

Add ignore button to the MacInterpreterSelectedAndNoOtherInterpretersDiagnostic & MacInterpreterSelectedAndHaveOtherInterpretersDiagnostic diagnostic message to provide the ability to opt out of the warning. (Fixing related issue #4448)

* Generalize test results service (#4813)

* Generalize test results service

* News entry

* Fix gulp errors

* Updated status icons

* Added tests

* fix names for run above and run below code lenses (#4879)

* Fix multiline comments with text on first line. (#4884)

* Cherry-pick 26a7b9c

* cherry pick from master branch

* Revert "Revert "Always use the same jedi environment (#4687)" (#4850)"

This reverts commit 0268ef4.

* Update change log entries

* Bump ptvsd to 4.2.5 (#4904)

* Bump version of PTVSD to 4.2.5
* Changes to capabilities

* Ensure output panel does not steal focus due to ls errors (#4919)

For #4868

<!--
  If an item below does not apply to you, then go ahead and check it off as "done" and strikethrough the text, e.g.:
    - [x] ~Has unit tests & system/integration tests~
-->
- [x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
- [x] Title summarizes what is changing
- [x] Has a [news entry](https://github.com/Microsoft/vscode-python/tree/master/news) file (remember to thank yourself!)
- [n/a] Has sufficient logging.
- [n/a] Has telemetry for enhancements.
- [n/a] Unit tests & system/integration tests are added/updated
- [n/a] [Test plan](https://github.com/Microsoft/vscode-python/blob/master/.github/test_plan.md) is updated as appropriate
- [n/a] [`package-lock.json`](https://github.com/Microsoft/vscode-python/blob/master/package-lock.json) has been regenerated by running `npm install` (if dependencies have changed)
- [n/a] The wiki is updated with any design decisions/details.

* Fix change log

* Bump pinned version of Language Server

* Release for march 2018 (#4931)

* New version
* Updated change log

* Same logic for stable and beta (#4964)

* Point release for March 2018 (#4965)

* Capture telemetry when tests are disabled (#4997)

For #4801

* Update change log
DonJayamanne added a commit that referenced this pull request Apr 4, 2019
* release:
  March 2019 point release with debugger fixes (#5056)
  Update change log
  Capture telemetry when tests are disabled (#4997)
  Point release for March 2018 (#4965)
  Same logic for stable and beta (#4964)
  Release for march 2018 (#4931)
  Bump pinned version of Language Server
  Fix change log
  Ensure output panel does not steal focus due to ls errors (#4919)
  Bump ptvsd to 4.2.5 (#4904)
  Update change log entries
  Revert "Revert "Always use the same jedi environment (#4687)" (#4850)"
  cherry pick from master branch
  Cherry-pick 26a7b9c
  Fix multiline comments with text on first line. (#4884)
  fix names for run above and run below code lenses (#4879)
  Generalize test results service (#4813)
  Add ignore button to the MacInterpreterSelectedAndNoOtherInterpreters… (#4808)
  Pin to beta version of PTVSD (#4836)
  Update tpn distro and 3rd party notices
@lock lock bot locked as resolved and limited conversation to collaborators Jul 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants