-
Notifications
You must be signed in to change notification settings - Fork 8
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
Specify folder to use #26
Comments
Ok, so I thought I could pass the sudo: false
language: node_js
cache:
directories:
- node_modules
notifications:
email: false
node_js:
- '9'
before_script:
- npm prune
script:
- npm run build
# - cd public
- npm i -g now-pipeline
- touch .npmignore
- DEBUG=now-pipeline now-pipeline /public/
# - now-pipeline --test "npm run deploy"
# - now-pipeline-list
# - npm i -g now-pipeline
# - now-pipeline-prune
# - now-pipeline 'npm run deploy'
# - now-pipeline --alias scottspence.now.sh
# after_success: And I have the output for it here it seems to be deploying the whole project, well that's what I'm gleaning from now-pipeline deploying 480 files: 4.97s$ DEBUG=now-pipeline now-pipeline /public/
Mon, 08 Jan 2018 10:53:10 GMT now-pipeline deploying 480 files
Mon, 08 Jan 2018 10:53:10 GMT now-pipeline [ 'package.json',
.... |
Yea @bahmutov Is this something you'd be interested in supporting? I believe I have a way to add this feature with very minimal changes. I have a pretty solid grasp on the flow of this but obviously you know more than I do. From the zeit docs this seems like it would be valid. So if you want I can put up another PR so you can my thoughts. If not just let me know. |
Adding a folder option would be great!
…Sent from my iPhone
On Jan 8, 2018, at 15:13, Joseph Snell ***@***.***> wrote:
Yea now-pipeline is not the same as the cl tool.
@bahmutov Is this something you'd be interested in supporting? I believe I have a way to add this feature with very minimal changes. I have a pretty solid grasp on the flow of this but obviously you know more than I do. From the zeit docs this seems like it would be valid. So if you want I can put up another PR so you can my thoughts. If not just let me know.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Added a new PR for your review and changes! Open to all thoughts. |
Hi @wulfmann I've tried this with So from the error message and looking at the PR merged does the arg need to have the I'm now going to try here: https://travis-ci.org/spences10/blog.scottspence.me/builds/327671010 |
No none of the variations does anything when specifying a directory it always results in the previously pictured error message |
I will take a look at this tonight. Is there any chance you could share your directory structure and package.json so I can recreate? |
Hi @wulfmann, I changed over to the zeit example eventually, folder structure is here, and here is my The Thanks |
@spences10 Yes that would be great if you could close this one. I'm still looking into the other issue with the deploy, but since this is isolated and you feel good about it, go ahead and close. |
Hi, I'm using the
- DEBUG=now-pipeline now-pipeline
script trying to get to grips with this toolFrom the output here:
I'm still not completely clear on what the command
now-pipeline
does, because the output appears to be reading from the root of the project and I want to only deploy thepublic
folder in my Gatsby project I'm cd'ing to that folder then running thenow-pipeline
command and getting errors because there's nopackage.json
🤷♂️There shouldn't be a
package.json
it's a static site, does that need to be configured as well?It's not the clearest on how to use, even with the
README.md
The text was updated successfully, but these errors were encountered: