-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Add PyEnvInfo and the other base info types for the Python environments component. #13708
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 PyEnvInfo and the other base info types for the Python environments component. #13708
Conversation
/** | ||
* IDs for the various supported Python environments. | ||
*/ | ||
export enum PythonEnvKind { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will this be replacing EnvironmentType
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes (eventually)
Codecov Report
@@ Coverage Diff @@
## master #13708 +/- ##
=======================================
Coverage 59.80% 59.80%
=======================================
Files 674 675 +1
Lines 37758 37796 +38
Branches 5438 5445 +7
=======================================
+ Hits 22581 22604 +23
+ Misses 14021 14017 -4
- Partials 1156 1175 +19
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few comments, we can also address some of these when working on the collection if you prefer that.
Kudos, SonarCloud Quality Gate passed!
|
[ ] 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 runningnpm install
(if dependencies have changed).[ ] The wiki is updated with any design decisions/details.