-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Build: new notification system for build notifications #10598
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
Comments
Also noted from today, latex pdf builds could emit a warning when we get a non-zero exit code. |
I reviewed all the issues linked and wrote down some thoughts that I pasted at https://gist.github.com/humitos/e9782439bee9ad6c5897a5557c163244 This is not the final version, but just some ideas / concepts and notes that I took while reading all those issues. We have a lot of duplicated information around and I wanted to summarize them all in just one document. |
This came from a discussion at our offsite. I put up an internal discussion at https://github.com/readthedocs/meta/discussions/126 in case there is anything left to discuss. But if not, starting with the build notification changes is probably where we'd start with this work I imagine.
To distill this conversation heavily:
We want to overhaul the way we output notifications and messages from the build. This would replace:
Instead:
#3399 was the original start of this, but I hesitate to reuse that issue as it was used for multiple changes already. Perhaps it's best to close that up and move the relevant discussion.
The text was updated successfully, but these errors were encountered: