Skip to content

build(devcontainer): remove deprecated extension#157

Merged
CuriousDolphin merged 4 commits into
FocoosAI:mainfrom
B-AROL-O:gmacario/fix-issue-156
Sep 26, 2025
Merged

build(devcontainer): remove deprecated extension#157
CuriousDolphin merged 4 commits into
FocoosAI:mainfrom
B-AROL-O:gmacario/fix-issue-156

Conversation

@gmacario
Copy link
Copy Markdown
Contributor

Fix #156

@gmacario gmacario marked this pull request as draft September 24, 2025 11:35
@CuriousDolphin CuriousDolphin marked this pull request as ready for review September 24, 2025 15:37
@gmacario
Copy link
Copy Markdown
Contributor Author

gmacario commented Sep 24, 2025

NOTE: I am keeping this PR in draft since I realized there are three devcontainer.json in the repository and I haven't yet had the time to check whether bug #156 affects also the other files

@gmacario gmacario force-pushed the gmacario/fix-issue-156 branch from 37cd2af to 9daf17d Compare September 25, 2025 06:02
@gmacario
Copy link
Copy Markdown
Contributor Author

NOTE: I am keeping this PR in draft since I realized there are three devcontainer.json in the repository and I haven't yet had the time to check whether bug #156 affects also the other files

Done. @CuriousDolphin please verify

@gmacario
Copy link
Copy Markdown
Contributor Author

I am scratching my head to understand why 9 tests out of 10 are failing, and also GitHub Copilot seems puzzled:

image

Anyone from the maintainers can give me a hint?

@CuriousDolphin
Copy link
Copy Markdown
Member

I am scratching my head to understand why 9 tests out of 10 are failing, and also GitHub Copilot seems puzzled:

image Anyone from the maintainers can give me a hint?

Don't worry, those logs in the tests are normal; they verify edge conditions. The tests fail due to a problem (probably with GitHub permissions) with the action that writes the summary comment on the tests, but the tests were performed correctly!
We are investigating to fix the action.

Screenshot 2025-09-25 alle 12 36 26

@CuriousDolphin CuriousDolphin merged commit 0d06187 into FocoosAI:main Sep 26, 2025
1 of 10 checks passed
@gmacario gmacario deleted the gmacario/fix-issue-156 branch September 26, 2025 13:44
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.

bug(devcontainer): Deprecated extension "pylens" detected

2 participants