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

Tree view bugfixes #88

Merged
merged 6 commits into from
Nov 22, 2023
Merged

Tree view bugfixes #88

merged 6 commits into from
Nov 22, 2023

Conversation

deribaucourt
Copy link
Member

@deribaucourt deribaucourt commented Nov 22, 2023

No description provided.

When adding a new recipe to the workspace, the recipe name is now
trimmed to remove the version number and the extension. This helps user
properly format their requests.
The settings are redefined in most integration tests. However, they are
now fixed when opening the remote SSH demo.
Despite the correct file path being passed to the vscode.open command,
executing it in a remote SSH environment would try opening the file
locally. Converting to a vscode.Uri in the SSH execution context does
the trick.
@deribaucourt deribaucourt changed the title Fix: Trim recipe name when adding new recipes to the workspace Tree view bugfixes Nov 22, 2023
Parsing errors where improperly reported in several situations. For
example when starting the extension, before any scan has happened yet.

This can lead to cases where 0 recipes have been scanned but it is
properly reported, and clicking the bar will trigger a rescan.
When a bitbake file was saved while the parse all recipes task was
already running, this resulted in a warning message being displayed.
@deribaucourt deribaucourt marked this pull request as ready for review November 22, 2023 14:25
@deribaucourt deribaucourt merged commit 5ef1027 into staging Nov 22, 2023
3 checks passed
@deribaucourt deribaucourt deleted the view-fixes branch November 22, 2023 15:51
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.

2 participants