pepkit / looper

A job submitter for Portable Encapsulated Projects
http://looper.databio.org
BSD 2-Clause "Simplified" License
20 stars 7 forks source link

Adding pydantic argparse cli and Refactor Looper Tests #472

Closed donaldcampbelljr closed 3 months ago

donaldcampbelljr commented 4 months ago

This PR accomplishes these major tasks:

CLI

TODO:

donaldcampbelljr commented 4 months ago

Some misc notes on usage by example: looper.cli_pydantic init --force-yes looper.cli_pydantic --looper-config .looper_project.yaml runp looper.cli_pydantic --looper-config .looper.yaml rerun

donaldcampbelljr commented 4 months ago

I had to add init-piface as init_piface. Will revisit.

donaldcampbelljr commented 4 months ago

Commands are added.

donaldcampbelljr commented 4 months ago

These have been refactored and now work with manual testing and some of the original pytest suite.

donaldcampbelljr commented 4 months ago

The original CLI is now removed as well.

nsheff commented 3 months ago

I think you're collecting other changes on this PR -- I think you may need to rebase the dev branch into feature-pydantic-argparse-cli, because the incoming branch here is carrying with it a bunch of unrelated updates, making it hard to review the PR

donaldcampbelljr commented 3 months ago

I merged the downstream changes into target branch to make it a bit more legible.

donaldcampbelljr commented 3 months ago

Partially addresses https://github.com/pepkit/looper/issues/464, though there still is work to be done with writing more comprehensive smoketests