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

Compilation in progress feedback #614

Closed
ncannasse opened this issue Dec 8, 2023 · 21 comments · Fixed by #615
Closed

Compilation in progress feedback #614

ncannasse opened this issue Dec 8, 2023 · 21 comments · Fixed by #615
Assignees

Comments

@ncannasse
Copy link
Member

When upgrading to Haxe development, I no longer get "Building..." during compilation in VSCode
That's quite disturbing :-)
image

@Simn
Copy link
Member

Simn commented Dec 8, 2023

I still see "Building Cache":

Code_KZ5Qk3zZZc

I'm not sure what a generic "Building" actually is/was.

@ncannasse
Copy link
Member Author

When you Shift+Ctrl+B to compile

@Simn
Copy link
Member

Simn commented Dec 8, 2023

Ah yes, that I can confirm. Will have to check what changed here.

@ncannasse
Copy link
Member Author

Ty if you can fix this, it's very hard now to tell if the build is ongoing or not

@Simn
Copy link
Member

Simn commented Dec 8, 2023

I'm not seeing that anymore when building Haxe itself either, so this might be some general VS Code thing. But I agree it's quite confusing to not have a clear indicator anywhere.

@kLabz
Copy link
Contributor

kLabz commented Dec 8, 2023

When upgrading to Haxe development

But this implies it still works with earlier Haxe versions?
Will try to repro tonight

@Simn
Copy link
Member

Simn commented Dec 8, 2023

I can't see how this would be related to Haxe versions unless vshaxe has some if (haxeVersion == 5) doSomethingDifferent() code somewhere...

@kLabz
Copy link
Contributor

kLabz commented Dec 8, 2023

Adding startProgress("Building"); before that line seems to do the trick; not sure what changed / what was doing that before.

Edit:
Could be linked to microsoft/vscode#192925 or microsoft/vscode#190987
Might need to update our tasks

@ncannasse
Copy link
Member Author

I confirm btw that this is related to Haxe version, as if I switch between our Shiro branch (somewhere around ~4.2.5) and development the Building... disappear. Would be be great to have it fixed and pushed to a vshaxe release as we're planning to upgrade Haxe at Shiro very soon.

@kLabz kLabz self-assigned this Dec 9, 2023
@kLabz
Copy link
Contributor

kLabz commented Dec 9, 2023

Ok I know what's going on: vscode only shows the indicator if a task has problem matchers, and vshaxe doesn't use those with Haxe >= 4.3.

@Simn
Copy link
Member

Simn commented Dec 10, 2023

Keeping this open as a reminder to look into it again at some point, because the current solution doesn't seem quite right.

@Simn Simn reopened this Dec 10, 2023
@ncannasse
Copy link
Member Author

When can we expect a VSHaxe release @Simn @kLabz ? :)

@Simn
Copy link
Member

Simn commented Dec 16, 2023

I don't think I have the required access for making releases. IIRC the right person to ping here is @AlexHaxe.

@AlexHaxe
Copy link
Member

AlexHaxe commented Dec 16, 2023

Github actions will automatically release a new version when you tag a version number to a commit (obviously the new version number should already be in package.json at that point).

there is a chance our marketplace token expired, so we might need nadako to update it and then rerun the failed Publish task.

@ncannasse
Copy link
Member Author

Up :) @nadako

@nadako
Copy link
Member

nadako commented Dec 18, 2023

Hm, I'm pretty sure I was renewing the token recently, can you guys try?

@Simn
Copy link
Member

Simn commented Dec 18, 2023

Oh I somehow thought Alex said it wasn't working. I just made a tag, let's see!

@Simn
Copy link
Member

Simn commented Dec 18, 2023

Well, looks like that wasn't it...

[publish](https://github.com/vshaxe/vshaxe/actions/runs/7253264612/job/19759499365#step:4:151)
Error: nadako.vshaxe v2.30.0 already exists.
    at _publish (/home/runner/work/_actions/HaaLeo/publish-vscode-extension/v1/dist/index.js:3630:27)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async publish (/home/runner/work/_actions/HaaLeo/publish-vscode-extension/v1/dist/index.js:3568:13)

@AlexHaxe
Copy link
Member

AlexHaxe commented Dec 18, 2023

sorry, was out sick the last few days.

for a new version version field in package(-lock).json needs to match and potentially submodule(s) need to be updated and maybe patch notes. then it should work (my read from the rejection message is that the token should still be valid)

@Simn
Copy link
Member

Simn commented Dec 18, 2023

Alright, that did it. Thank you for the instructions!

@Simn Simn closed this as completed Dec 18, 2023
@LeoVgr
Copy link

LeoVgr commented Feb 2, 2024

I still got this issue with the default tasks.json on vshaxe 2.31. Looks like the problemMatcher entry is the problem here because when deleted, the "Building ..." reappear.

I don't know if this is linked to this problem, but the haxe-fake problem matcher isn't recognized too :
Unrecognized problem matcher. Is the extension that contributes this problem matcher installed?

Without the tasks.json file, the building feedback shows up.

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 a pull request may close this issue.

6 participants