Skip to content

provide a shorter alias for workingDirectory#303

Merged
devblackops merged 1 commit intopsake:masterfrom
SeidChr:patch-1
Sep 18, 2020
Merged

provide a shorter alias for workingDirectory#303
devblackops merged 1 commit intopsake:masterfrom
SeidChr:patch-1

Conversation

@SeidChr
Copy link
Copy Markdown
Contributor

@SeidChr SeidChr commented Jul 16, 2020

to enable more compact build tasks

Description

adding the alias "wd" for the "workingDirectory" parameter of the Exec command.

Motivation and Context

It helps putting short commands in a single line and still keep them readable.
Especially usefull when putting many tasks below each other in a tabular form

How Has This Been Tested?

The change is minimal intrusive, and using a default Powershell feature. Not adding any testable logic.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

to enable more compact build tasks
@stale
Copy link
Copy Markdown

stale bot commented Sep 17, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Sep 17, 2020
@SeidChr
Copy link
Copy Markdown
Contributor Author

SeidChr commented Sep 17, 2020

it was far more effort to write the description than it was to write that single line change.
now it's stale after 2 month... what is blocking this?

@stale stale bot removed the stale label Sep 17, 2020
@devblackops
Copy link
Copy Markdown
Member

@SeidChr Nothing blocking this. It just slipped past me amongst the million GH notifications I get.

This LGTM. Sorry for not responding sooner.

@devblackops devblackops merged commit 41c466c into psake:master Sep 18, 2020
@SeidChr
Copy link
Copy Markdown
Contributor Author

SeidChr commented Sep 18, 2020

thanks ! 😀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants