-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Access public history in job cache / job search #19108
Access public history in job cache / job search #19108
Conversation
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.
Looks fine. SQLite fix makes sense.
Thanks!
Ready to move out of draft?
717eb97
to
360b72a
Compare
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.
This is great, thanks!
🎉 |
This seems like it really needs to be accompanied with some sort of tool syntax that declares the tool depends on the user running the job 😓. |
I haven't understood the link here, what would the tool annotation say ? |
I think this is ready, but maybe want to make it configurable on the user side if we want to pick up public results ?
How to test the changes?
(Select all options that apply)
License