-
Notifications
You must be signed in to change notification settings - Fork 338
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: file support for harness #720
Conversation
a80139c
to
b40db35
Compare
b40db35
to
440c6a1
Compare
Co-Authored-By: Karol Maciaszek <[email protected]>
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.
Minor improvement
914f206
to
88c44c8
Compare
Nice to see comments that otherwise wouldn't have popped up from the gigantic PR by the way :) |
88c44c8
to
5f80046
Compare
Co-Authored-By: Karol Maciaszek <[email protected]>
5f80046
to
503e6d7
Compare
@XVincentX changelog line is missing |
It was on purpose, since we do not want to document internal changes that do not affect the users (library updates and shit). However this might affect users writing tests. Adding a line. |
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 has been back ported from #669
By merging this in master the diff of the Proxy PR shall become significantly smaller and help people get that 馃悏 over the finish line.