Added ability to specify target of build - #1043
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1043 +/- ##
=======================================
Coverage 45.09% 45.10%
=======================================
Files 155 155
Lines 62701 62705 +4
=======================================
+ Hits 28278 28280 +2
- Misses 34423 34425 +2
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
Hey, I don't mean to be pushy or anything like that. I just wanted to say that this PR is ready for review and let me know if I need to change anything. This is my first OSS contribution, so please excuse my stupidity if I did something wrong. |
|
@xinslu Sorry, this is all on my end — I've been overwhelmed with work responsibilities lately and am far behind on Tectonic reviews. This sounds like a change that I would like to accept so I aspire to look at it soon! And please feel free to ping me just in case I lose track of things. I try not to do that, but with so much going on it can easily happen. |
This is slightly more efficient. Also delete the comment, because the intent of the relevant code is clear enough to not need explanation.
|
I've made a few tweaks here — since I've been taking so long to look at this, I've gone ahead and pushed them to your branch so that I can go ahead and merge once the CI checks pass. Thanks for your contribution! |
2 participants
Solves #970.
Simple PR that checks if an optional target is specified in the build command. If yes, matches it and builds only the specific output name.