-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Space in extension path: Make testing part of insiders.yml/release.yml #14168
Comments
@rchiodo Do we need a separate PR for release.yml, or can we consider this done? |
I guess that's open for discussion? I think insiders testing of spaces is probably good enough. It should catch any errors before they get into the release branch. So I'd say this is done. But if you guys want to double check the release branch too, feel free to copy the changes to the release.yml. |
Update: we're going to deal with it now, and make similar changes to the release workflow. |
After all of these issues are fixed:
#14163
#14164
#14165
#14166
#14167
Submit this pr here (after modifying it to not run on that specific branch):
#14113
The text was updated successfully, but these errors were encountered: