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
Is this project dead? #360
Comments
Looks it. But I'm seeing a lot of activity over here: https://github.com/Lightricks/xcpretty @byohay @barakwei you look as though you are doing some maintenance on xcpretty, are you planning on putting out a release? Any chance we could get #345 or something similar merged into it if so? |
At this stage, we don't plan on releasing our own xpretty as a gem, nor do we currently accept external pull requests, since we tailor our fork to our needs. Since we maintain our own fork we can point the xcpretty gem to our git repo. In the If you're planning to use our fork, notice we've removed some features like custom formatters/reports and we intend to remove reports completely. |
@barakwei thanks for the info.
If xcodebuild played that nicely xcpretty would have zero reason for existence... my current warnings I want to silence are out of the linker, and swift, which you can't silence or at least last time I looked |
I see. I guess we would accept pull requests that are highly requested and have tests. |
That's a kind gesture @barakwei, but I think the solution here shouldn't involve simply "failing over" to an individual fork just because there happens to be activity on it. As you say, you're taking xcpretty in your own direction, and I don't think it's necessarily fair or right to now burden you with maintenance tasks for the mainline version that you've already left behind (design-wise). What I'm hoping to do here is attract the attention of the maintainers to add some new faces to the github org such that the project can continue to evolve. Or, to declare that the project is abandoned -- that we should come up with alternate plans for sorting out all the pull request contributions. |
Sure thing. I do recommend using our fork since the mainline version is very old and doesn't support many new commands and the new build system. Also, we have some bugfixes and performance improvements. |
Since nobody else has responded to address this issue, here’s my 2 cents: Since we’re using an in-house solution nowadays (and I don’t work in infra anymore), I don’t have time to keep up with Apple‘s changes and maintain the project. The project has always had > 1 contributors with commit access, but the truth is nobody has time and energy to properly maintain it if they’re not using it themselves. Since xcpretty is used in some bigger & more popular projects as a dependency, I’ve transferred the ownership as an alternative to shutting it down. As there’s no active maintainers to keep it up and running, IMO it’d be the best to deprecate it and move on with life. If there’s enough people like @barakwei that would like to take full ownership onwards, let me know and you’ll get commit access. Are you “pulling an Apple”?tl;dr - No. There’s better solutions nowadays. Xcode is also dumping various XML reports (e.g. for test runs), and there’s many tools like Fastlane’s trainer that can post process the test runs and print results nicely. It shouldn’t be hard to write your own micro processing tool when the data you have is structured. That’s OK, but we still need xcpretty for xyzLike mentioned before, leave comments below and I’ll happily consider letting more maintainers in. |
Is there a reason to not just hand out commit access to the people you feel are qualified to make edits? I can't think of a downside to having more people in a position to merge pull requests; there is a functioning CI and test suite, old versions of this project will always be available if anything goes wrong, and in the event that something does go off the rails, whatever focus that brings to this project would likely result in the bugs being resolved (and more people contributing). |
Not sure if it was clear from my previous comment, but there’s a few people with commit access in the repo (3 directly in xcpretty/xcpretty plus 4 via xcpretty org). This means that people were given access before, and lost interest over time. When that happens, all the issues, mentions etc default back to me. Adding yourself to maintainers is an option I’ve mentioned above. |
Sorry if the above comment sounded harsh, dumped my thoughts without much thinking. |
I've been thinking about this and I think I'm game to take this on. Is there a doc somewhere about "things we tried that were definitely bad ideas" and/or "purposeful design choices that should be preserved"? |
Sorry for the delay Ian. I can write up a doc and do a hangout to walk you through everything. |
There's no rush; plenty to keep me busy on my end. I'll stay tuned. |
@ianfixes completely missed doing this before, but in case you still need it, as promised: handover brain dumpContext on stdout/stderr and how xcpretty worksIn order to capture build errors in a tty, either one of these needs to happen:
One of xcodebuild's problems is that it prints some failures to stderr, One way of mitigating this is using tee to dump the raw log into a separate Lastly, because of this approach, we can't record both stdout and stderr Blacklist vs WhitelistIn the whitelist approach, xcpretty takes input line by line and pretty
If xcpretty knows how to parse this failure, it'll parse and print it out. This is why it's called whitelisting: it prints only the lines it knows The usage would look something like this: xcodebuild 2>&1 | xcpretty
# ^ ^
# | L No need for | tee xcodebuild.log
# |
# redirect stderr->stdout As Apple's toolchain constantly evolves, some of the outputs keep changing. Support default OSX rubyThe average person installing xcpretty is an iOS dev or a CI machine. What worked wellTestsThe project was written in a TDD fashion, which led to a testable and simple No dependenciesOne thing that definitely eased the pain of distributing xcpretty was the What didn't work so well (personal opinion)xcpretty started as a build log formatter and a "do one thing, but do it Swift?If I was writing something like xcpretty today from scratch, I'd probably |
can someone please publish a new gem that forks away from this repo so we can get some new development in please? Not a ruby developer but would really like to see some updates on xcpretty. |
Either that or add a new member to the organization who can start triaging all the PRs |
@ianfixes Haven't heard back from you from the writeup I did above - are you still interested? |
What's the current status on this @ianfixes? |
Based on employment, I'm unfortunately no longer in a position to take on a stewardship role in this project. But it does seem like the |
The last time a pull request was merged in this project was (as of this writing) almost exactly a year ago -- #335. The list of contributors doesn't show anything promising either, and their names don't seem to be among the comments in the most recently-updated pull requests.
Has the project been abandoned?
The text was updated successfully, but these errors were encountered: