-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fix #185 make the jrxml path absolute #190
Fix #185 make the jrxml path absolute #190
Conversation
Build running now! |
44e2d3d
to
814b042
Compare
ahhh its failing the formatter! |
814b042
to
cdf1a5f
Compare
formatted it now. |
import java.io.FileOutputStream; | ||
import java.io.IOException; | ||
import java.io.InputStream; | ||
import java.io.*; |
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.
Can these be re-expanded, please?
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.
That part was autoformatted by my Ide.
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.
I'll update the PR, after I see the builds pass.
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.
@bulldog98 if Ubuntu passes you are good. Its MacOS that takes forever so if Ubuntu passes its good enough not to wait.
Re-running |
cdf1a5f
to
289fb88
Compare
@all-contributors add @bulldog98 for code |
I've put up a pull request to add @bulldog98! 🎉 |
Fix #185