Skip to content
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

Automated dev terminal recordings #4702

Closed
wants to merge 13 commits into from

Conversation

hexylena
Copy link
Member

@hexylena hexylena commented Feb 5, 2024

not sure if we should ship this as is

it's a nice improvement, since it gives the user somehing slightly more interactive to play with. but it also should be made more general. And perhaps it's a slight downgrade from xml syntax highlighting? Alternatively also should run the planemo command itself (using more of the auto-gat machinery, rather than just 'typing' the first file.)

it's an improvement over the existing auto-gat pipeline since it's just a cast file, we can have the CI regenerate these on-demand, and update the tutorial.cast file next to the tutorial and it all works nicely.

asci

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reported by reviewdog 🐶
A code_in was opened, but closed with code_out on line 432

@@ -435,6 +436,19 @@ Note that for using `planemo`from a new shell you will need to activate the pyth
> 3. `planemo SUBCOMMAND --help` will show the usage information for the corresponding subcommand. Try to obtain the information for the `lint` subcommand.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reported by reviewdog 🐶
A code_in was opened, but closed with hands_on on line 436

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [rdjsonl] <GTN:028> reported by reviewdog 🐶
You have skipped a heading level, please correct this.

Listing of Heading Levels
# Galaxy Tools and Conda
## Writing a Bioconda Recipe
## Building the recipe locally
## Creating a Pull Request
## Installing a Conda Package
# Galaxy Tool Wrappers
## Installing Planemo
## Pre-recorded demo
## Initializing a Tool Wrapper
## Galaxy Tool Wrappers
### Tool section
### Description section
### Macros section
### Bio.tools ID
### Requirements section
### Input section
#### *Param*
#### *Conditional*
#### *Section*
#### *Inputs Section in Practice*
### Outputs section
#### *Filtering the output*
#### *Output collections*
#### *Discover datasets*
### Command section
#### *Cheetah commands*
# if ${parameter} == "value":
# end if
# if ${parameter} == "value":
# else:
# end if
# if ${parameter} == "value":
# else if ${parameter} == "other value":
# else:
# end if
# for value in ${param}
# end for
# set $variable = value
#### *Using other scripts*
#### *In practice*
#### Symlinks
### Test section
#### *Collections and sections in tests*
### Help section
### Citations section
## Final wrapper
## Toolshed file
# Testing Galaxy tool with `planemo`
## Using `planemo` to lint tools
## Using `planemo` to test tools
## Using `planemo` to serve tools
# Publishing Galaxy tools
## Fork and clone the github repository
## Create a pull request for a new tool
## The IUC style continuous integration workflows
## Create your own IUC style tool repository
# Conclusion
reviewdog suggestion errorsource lines are not available

@hexylena hexylena closed this Dec 5, 2024
@hexylena
Copy link
Member Author

hexylena commented Dec 5, 2024

it's a nice feature, but, no time.

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

Successfully merging this pull request may close these issues.

1 participant