-
-
Notifications
You must be signed in to change notification settings - Fork 392
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
Not super helpful error messages #3635
Comments
It seems that ReSpec (rather than Pubrules) emitted the error. @marcoscaceres Can I get a pointer to more information on this error? |
The problem seems to have been fixed. Closing. |
Thanks @xfq! |
Sorry, I totally missed this! Glad it was resolved. |
Oh! about this error... sometimes, publishing to TR more than once per day fails. It's a known spec-prod issue. If it happens, rerunning the action again sometimes fixes it... worst case, run the action again the next day and it works 😇 |
Thanks for the tip, that explains why this was hard to repro. It'd be nice if this could fail with an informative error message. Meanwhile, let's say one TR publication a day is a spec-prod feature and not a bug 😀 |
I think this was the original:
We should probably reword that as:
And the Plugin should be moved last. The hint test is ok, I think. @anssiko, wdyt? Or are the errors not helpful at all? |
Reopening and moving to ReSpec... |
@anssiko, any "real user" feedback you can provide us would be super helpful. Will do our best to improve things. |
I was actually reacting to "publishing to TR more than once per day fails" case. I was wondering if spec-prod could detect that and warn the operation might fail, if that's the case. Optimally, it wouldn't fail. |
Ah, it does report a fail, but the errors are also quite cryptic: |
That shouldn't be happening. It's probably some other bug where spec-prod didn't set right right respecConfig. Will investigate. |
We did a bunch of work on this in #3637. Closing for now and we can fix up any specifically unhelpful ones. |
pr-push.yml fails with:
via https://github.com/w3c/device-posture/runs/2727557242?check_suite_focus=true
@xfq
The text was updated successfully, but these errors were encountered: