Skip to content

Plugin to view .nextflow.log files using toolong #2895

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

Draft
wants to merge 13 commits into
base: dev
Choose a base branch
from

Conversation

ewels
Copy link
Member

@ewels ewels commented Mar 22, 2024

⚠️ Relies on Textualize/toolong#47
(Ok technically it builds from my forked dev branch of the above, but it shouldn't do long term)

Could do with some refinement. Specifically:

  • Don't just nuke all toolong functionality, only switch on when we think we see Nextflow logs
  • Figure out weird behaviour with error / warning / debug blocks (probably some async behaviour?)
  • Create command alias, eg. nf-core log or similar.
  • Write docs

Testing: pip install this PR and try nf-core log in a directory where you've run Nextflow.

This comment was marked as off-topic.

@ewels ewels changed the title Add plugin entrypoints for toolong Plugin to view .nextflow.log files using toolong Mar 24, 2024
@ewels ewels marked this pull request as ready for review March 25, 2024 23:33
@ewels ewels marked this pull request as draft April 3, 2024 14:39
@ewels ewels added the WIP Work in progress label Apr 3, 2024
@mirpedrol
Copy link
Member

🧹 spring cleaning message 🌷

Is this PR still relevant? Should we work on finishing it up?

@ewels
Copy link
Member Author

ewels commented Mar 18, 2025

It stalled pretty bad. Although I really like how it works, the fundamentals are kind of broken (not being able to have contextual multi-line log level highlighting).

It's on my backlog to revisit this, probably rewriting from the ground up without toolong and using a totally custom Textual TUI 😢 I'll almost certainly use this as a reference, if not building from it, so I'd prefer to leave it open if possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants