Skip to content

Func plus hooks#578

Merged
kmvanbrunt merged 6 commits intomasterfrom
func_plus_hooks
Oct 13, 2018
Merged

Func plus hooks#578
kmvanbrunt merged 6 commits intomasterfrom
func_plus_hooks

Conversation

@kmvanbrunt
Copy link
Copy Markdown
Member

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 13, 2018

Codecov Report

Merging #578 into master will increase coverage by <.01%.
The diff coverage is 88.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #578      +/-   ##
==========================================
+ Coverage   93.18%   93.19%   +<.01%     
==========================================
  Files          11       11              
  Lines        3185     3188       +3     
==========================================
+ Hits         2968     2971       +3     
  Misses        217      217
Impacted Files Coverage Δ
cmd2/pyscript_bridge.py 97.81% <88.88%> (+0.03%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3dcff64...be82ee5. Read the comment docs.

CHANGELOG.md Outdated
* All platforms now depend on [wcwidth](https://pypi.python.org/pypi/wcwidth) to assist with asynchronous alerts.
* Macros now accept extra arguments when called. These will be tacked onto the resolved command.
* All cmd2 commands run via py now go through onecmd_plus_hooks.
* Bug Fixes
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Put bug fixes at top


def _exec_cmd(cmd2_app, func: Callable, echo: bool) -> CommandResult:
"""Helper to encapsulate executing a command and capturing the results"""
def _exec_cmd(cmd2_app, command: str, echo: bool) -> CommandResult:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These changes seem good

tleonhardt
tleonhardt previously approved these changes Oct 13, 2018
@@ -1,7 +1,11 @@
## 0.9.6 (TBD)
## 0.9.6 (October 13, 2018)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good

@kmvanbrunt kmvanbrunt merged commit dcb5be4 into master Oct 13, 2018
@kmvanbrunt kmvanbrunt deleted the func_plus_hooks branch October 13, 2018 22:23
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