Skip to content

Comments

PR: Fixes and improvements for final integration with Spyder#36

Draft
ccordoba12 wants to merge 10 commits intospyder-ide:mainfrom
ccordoba12:fixes-for-remote-development
Draft

PR: Fixes and improvements for final integration with Spyder#36
ccordoba12 wants to merge 10 commits intospyder-ide:mainfrom
ccordoba12:fixes-for-remote-development

Conversation

@ccordoba12
Copy link
Member

@ccordoba12 ccordoba12 commented Jul 9, 2025

  • Fix importing env from bytes in Pixi backend.
  • Add create_kernelspec method to the manager (needed for remote development).
  • Return environments list in the format expected by Spyder (Pixi backend only for now).

@ccordoba12 ccordoba12 added this to the v0.2.0 milestone Jul 9, 2025
@ccordoba12 ccordoba12 self-assigned this Jul 9, 2025
@ccordoba12 ccordoba12 added the bug Something isn't working label Jul 9, 2025
@ccordoba12 ccordoba12 changed the title PR: Some fixes for remote development PR: Some fixes for remote environments Jul 9, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jul 9, 2025

Codecov Report

❌ Patch coverage is 43.92523% with 60 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.68%. Comparing base (37507ec) to head (859a31a).

Files with missing lines Patch % Lines
envs_manager/backends/pixi_interface.py 40.90% 50 Missing and 2 partials ⚠️
envs_manager/backends/conda_like_interface.py 40.00% 6 Missing ⚠️
envs_manager/manager.py 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   71.22%   69.68%   -1.55%     
==========================================
  Files          11       11              
  Lines        1199     1237      +38     
  Branches      132      133       +1     
==========================================
+ Hits          854      862       +8     
- Misses        289      319      +30     
  Partials       56       56              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ccordoba12 ccordoba12 force-pushed the fixes-for-remote-development branch from 90a5d24 to d5b082d Compare July 9, 2025 06:09
@ccordoba12 ccordoba12 force-pushed the fixes-for-remote-development branch from d5b082d to 47231b3 Compare July 9, 2025 06:20
@ccordoba12 ccordoba12 changed the title PR: Some fixes for remote environments PR: Some fixes and improvements for final integration with Spyder Oct 8, 2025
@ccordoba12 ccordoba12 changed the title PR: Some fixes and improvements for final integration with Spyder PR: Fixes and improvements for final integration with Spyder Oct 8, 2025
@ccordoba12 ccordoba12 force-pushed the fixes-for-remote-development branch 2 times, most recently from ca9e4c9 to 545c829 Compare October 8, 2025 06:15
This is necessary so that the version can be displayed in the UI after
creating an env.
@ccordoba12 ccordoba12 force-pushed the fixes-for-remote-development branch from 545c829 to a195102 Compare October 8, 2025 06:26
We need this to also use it on the UI side.
@ccordoba12 ccordoba12 marked this pull request as draft October 8, 2025 17:52
That's necessary to be able to use the same env name in a future
attempt.
That prevents showing the error report dialog in Spyder when an error
occurs.
Also, increase timeout for Windows CI so tests pass
That's more robust to failures on Windows.
@ccordoba12 ccordoba12 force-pushed the fixes-for-remote-development branch from 979b49e to 859a31a Compare October 23, 2025 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants