-
-
Notifications
You must be signed in to change notification settings - Fork 288
Cli reader options #1860
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
base: master
Are you sure you want to change the base?
Cli reader options #1860
Conversation
You are modifying libf3d public API! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1860 +/- ##
==========================================
+ Coverage 95.75% 95.89% +0.14%
==========================================
Files 123 127 +4
Lines 10612 11062 +450
==========================================
+ Hits 10161 10608 +447
- Misses 451 454 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Do you need any help or review to move forward @0xfedcafe ? |
Just that upper question about an optional and I’ll push hopefully the final commit for this task with the test |
Hi @0xfedcafe How is it going ? Do you need any help ? |
Hi @0xfedcafe Please do not hesitate to reach out if you need any guidance :) |
Hi @0xfedcafe ! Do you need a review or help with CI ? |
yes, i think a hint would be very helpful. i can't find why in some tests where the necessary plugins are loaded and the reader is not nullptr it still fails with "unsupported file format" |
fixes by mwestphal Co-authored-by: Mathieu Westphal <mathieu.westphal@gmail.com>
There a small conflict to fix with master, let me know if you need help with that |
Hi @0xfedcafe Do you need help moving forward ? |
Hi @0xfedcafe Do you need help moving forward ? |
Hey, yes, I would like to know whether I'm allowed to simply do a disjunction of the parts of error messages in failing test in my test as a regexp or that would be a bad solution? Because I'm unsure why it fails on different platforms with different error messages and I had no access to Linux/Windows so I couldn't test that. |
I'm confused a bit by your question, lets discuss on discord ? :) |
Added a CLI reader option #1735