-
Notifications
You must be signed in to change notification settings - Fork 16
Styling & theme tweaks #129
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
base: main
Are you sure you want to change the base?
Conversation
I'm getting as an error when I try to actually run this: nextflow run https://github.com/nextflow-io/hello -with-tower 70508760960023
|
This feature doesn't make much sense to me. It looks like you're trying to take a workflow run from seqera cloud and run the same pipeline locally. But then you need to make sure that the pipeline can be launched in the local environment with the requisite inputs, etc. It might work out-of-the-box for the You might need to go back to the drawing board on this one. @MythicalFish if you want to trim this PR down to the UI refactor (or make a separate PR) I will merge that |
Ok sounds good. Rip it out for now and just keep the bug CSS fixes in. @MythicalFish |
b2dde44
to
b90c8b3
Compare
this is good to merge from my POV. lmk @bentsherman |
Addresses:
Also refactors a bit and converts the custom icons to actual SVG