Skip to content

Cmd line bug fix#21

Merged
JarrodFolinoFC merged 2 commits intopatrickchugh:mainfrom
JarrodFolinoFC:cmd-line-bug-fix
May 29, 2023
Merged

Cmd line bug fix#21
JarrodFolinoFC merged 2 commits intopatrickchugh:mainfrom
JarrodFolinoFC:cmd-line-bug-fix

Conversation

@JarrodFolinoFC
Copy link
Copy Markdown
Collaborator

@JarrodFolinoFC JarrodFolinoFC commented May 29, 2023

YOUR PR TITLE

Type of Change

[ ] Bug Fix
[ ] New Feature
[ ] Refactor
[ ] Documentation

Checklist

All Submissions:

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you written Documentation/Tests?
  • Have you read done your own code-review?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully run tests with your changes locally?

Comment thread terravision
default="architecture",
help="Filename for output diagram (default architecture.dot.png)",
)
@click.option("--format", default="png", help="File format (png/pdf/svg/json/bmp)")
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

format is only for the draw command. You cannot draw json

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

cool

@JarrodFolinoFC JarrodFolinoFC merged commit 2c92bfc into patrickchugh:main May 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants