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
Fix extension marketplace #774
Conversation
- add loop to apply patches
fi | ||
fi | ||
done | ||
|
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.
That looks like a reasonable thing to include in any case - if you provide that as separate PR and have tested it I'd like to merge that now (if you can't do that yet).
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.
Yep. I've tested it.
|
So, shouldn't we close this and have microsoft/vscode#129360 included as patch instead? |
@GitMensch I think you have commented on the wrong issue... no? |
Totally. |
The I will revert the fix when CORS are correctly available on open-vsx.org. |
This PR is fixing #746 by reverting the CORS checks (added in
v1.58
).Additionally, it's adding a loop to apply patches.
@stripedpajamas If by the
v1.59.0
, the CORS haven't being added to openvsx, we should apply this PR.