Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings
Discussion options

nacho4d
Oct 8, 2021
Maintainer

I write some ideas of things I have noticed we need to work. More ideas, better ideas, just ideas are off course always welcome ;-]

Outstanding issues

Improvements

  • Consider moving some of the integration tests from ruby to swift. XCTestHTMLReport now has some basic support for integration tests and everything that is tested with nokogiri can be tested with NDHpple's XMLPath (at least in theory). I haven't investigated much but at least for tests we can stop relying on ruby.
  • Move to swift-argument-parser! (Already in the works by Tyler)
  • Move completely to Github Actions (Github actions are already being leveraged now we just need to stop Travis).
  • Improve the Wiki pages.

Investigation

  • SPM now supports resources so we could have the templates as text files rather than having them as static strings inside a swift file. Currently we release a binary and that is all. This is super convenient. If by moving to SPM resources users will need a binary + extra text files then maybe this change does not worth it.
    OR
  • Investigate about new extensible build tools (Swift 5.6) ... maybe we can keep the static strings approach but without using ruby scripts.
You must be logged in to vote

Replies: 4 comments · 1 reply

Comment options

nacho4d
Oct 11, 2021
Maintainer Author

@tylervick I share this with you. If you have more ideas feel free to add them here. :)
You must be logged in to vote
0 replies
Comment options

tylervick
Oct 11, 2021
Maintainer

Extend release action to update brew formula too.

I wasn't aware that this exists! We can probably remove the rb file in the main repo and make a downstream action after release to version bump.

Alternatively, we may be able to contribute directly to Homebrew/homebrew-core. Homebrew/homebrew-core#26977 was previously attempted but the repo is much larger now than it was at that time.
You must be logged in to vote
0 replies
Comment options

tylervick
Oct 11, 2021
Maintainer

Potential improvement: A webserver option in addition to the CLI. I.e. The ability to pass in xcresult bundles (either via filesystem or http upload) and output the generated html on the fly. I'm not sure how wide the use case is, but it might make sense given that these html bundles would often be uploaded to a (static) server anyway.
You must be logged in to vote
1 reply
@tylervick
Comment options

tylervick Oct 12, 2021
Maintainer

POC: https://github.com/tylervick/XCTestHTMLReport/pull/5/files
Comment options

tylervick
Oct 11, 2021
Maintainer

SPM now supports resources so we could have the templates as text files rather than having them as static strings inside a swift file

This is probably worth looking into! Or at the very least compressing these resources to reduce binary size a bit.

https://github.com/stencilproject/Stencil is another option. While it might be overkill for the existing CLI use case, it might tie in nicely with the http server idea I posted above.
You must be logged in to vote
0 replies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
💡
Ideas
Labels
None yet
2 participants
@nacho4d @tylervick
Morty Proxy This is a proxified and sanitized view of the page, visit original site.