Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Improve wireviz.parse() #250

Merged
merged 10 commits into from
Aug 5, 2022
Merged

Improve wireviz.parse() #250

merged 10 commits into from
Aug 5, 2022

Conversation

formatc1702
Copy link
Collaborator

Refactor and improve wireviz.parse().

  • More flexibility regarding supported types for inp argument
  • Allow multiple prepended files ($ wireviz -p prepend1.yml -p prepend2.yml mainfile.yml)
  • Customizable template separator; . stays as default
  • Cleanly distinguish between file output formats, and function return types
  • Better docstring

@formatc1702 formatc1702 mentioned this pull request Oct 16, 2021
Open
@formatc1702 formatc1702 mentioned this pull request Aug 5, 2022
Open
5 tasks
Base automatically changed from feature/isort-black to feature/cli August 5, 2022 15:51
@formatc1702 formatc1702 merged commit e31ed72 into feature/cli Aug 5, 2022
@formatc1702 formatc1702 deleted the feature/improved-parse branch August 5, 2022 16:06
@martinrieder martinrieder mentioned this pull request Jun 21, 2024
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@formatc1702