Skip to content

Add 'info' to set of commands#28

Open
adamrtalbot wants to merge 2 commits intoejseqera:devfrom
adamrtalbot:tw_info_cmd
Open

Add 'info' to set of commands#28
adamrtalbot wants to merge 2 commits intoejseqera:devfrom
adamrtalbot:tw_info_cmd

Conversation

@adamrtalbot
Copy link
Collaborator

@adamrtalbot adamrtalbot commented Jun 23, 2023

Additions:

  • info added to supported commands
  • Add example yml file in tests/

Changes:

  • the stdout is logged to console to see current status.

@adamrtalbot
Copy link
Collaborator Author

adamrtalbot commented Jun 23, 2023

Fixes #27

@adamrtalbot
Add 'info' to set of commands
a307467 
Additions:
 - info added to supported commands
 - Add example yml file in tests/

Changes:
 - the stdout is logged to console to see current status.
@adamrtalbot adamrtalbot requested a review from ejseqera June 27, 2023 13:04
@ejseqera ejseqera changed the base branch from main to dev June 28, 2023 17:59
@ejseqera
Copy link
Owner

ejseqera commented Jun 28, 2023

Thanks @adamrtalbot! I'm wondering if we want to run tw info by default before we parse the e2e yaml? Then we don't have to an empty info: key to the yaml. What do you think?

@adamrtalbot
Copy link
Collaborator Author

adamrtalbot commented Jul 17, 2023

I was thinking it would be explicit, i.e. only run info when specified in the YAML. We could use info: true?

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

Reviewers

@ejseqera ejseqera Awaiting requested review from ejseqera

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adamrtalbot @ejseqera