Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add working directory parameter #144

Merged
merged 2 commits into from
Jul 9, 2019
Merged

Add working directory parameter #144

merged 2 commits into from
Jul 9, 2019

Conversation

n6g7
Copy link
Contributor

@n6g7 n6g7 commented Jul 4, 2019

Adds a working_directory parameter to most commands and jobs.
Fixes #113 and #16.

@CLAassistant
Copy link

CLAassistant commented Jul 4, 2019

CLA assistant check
All committers have signed the CLA.

src/orb.yml Outdated Show resolved Hide resolved
@jennifer-shehane jennifer-shehane requested a review from bahmutov July 9, 2019 03:28
src/orb.yml Outdated
@@ -441,6 +471,20 @@ examples:
- cypress/run:
yarn: true

custom-directory:
description: |
Runs all commands in a custom directory.
Copy link
Contributor

Choose a reason for hiding this comment

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

ok, everything seems good, except can you describe this example a little bit better, please? Like the use case is monorepo with front-end tests in a subfolder for example ... right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes exactly, I've improved the description a bit, do you think that's good enough now?

src/orb.yml Outdated
@@ -391,6 +415,10 @@ jobs:
description: Use yarn to install NPM modules instead of npm.
type: boolean
default: false
working_directory:
description: Directory containing package.json
Copy link
Contributor

Choose a reason for hiding this comment

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

since this is top level parameter, could you describe its purpose a little bit more here?

Copy link
Contributor

@bahmutov bahmutov left a comment

Choose a reason for hiding this comment

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

I love it, just would like to see more description text before merging this change

@bahmutov
Copy link
Contributor

bahmutov commented Jul 9, 2019

perfect, thank you @n6g7

@bahmutov bahmutov merged commit 83a049a into cypress-io:master Jul 9, 2019
@n6g7 n6g7 deleted the working-directory branch July 9, 2019 16:27
@n6g7
Copy link
Contributor Author

n6g7 commented Jul 9, 2019

Cool! Do you plan on making a release soon? :)

@bahmutov
Copy link
Contributor

bahmutov commented Jul 9, 2019

@n6g7 I have just published v1.8.0 - give it a go

@n6g7
Copy link
Contributor Author

n6g7 commented Jul 10, 2019

Working great, thanks!

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.

Changing working directory
4 participants