Skip to content

Feature/hw 12#6

Open
Client-Z wants to merge 35 commits intodevelopfrom
feature/hw-12
Open

Feature/hw 12#6
Client-Z wants to merge 35 commits intodevelopfrom
feature/hw-12

Conversation

@Client-Z
Copy link
Copy Markdown
Owner

No description provided.

@serhiinkh
Copy link
Copy Markdown
Collaborator

I see npm ERR! missing script: db:migrate in the error logs you should add it to package json or remove this line from deploy.sh

@serhiinkh
Copy link
Copy Markdown
Collaborator

Also pushing deploy.sh to repository was bad idea because you have there sensitive data

@Client-Z
Copy link
Copy Markdown
Owner Author

I see npm ERR! missing script: db:migrate in the error logs you should add it to package json or remove this line from deploy.sh

@nicebro Thanks for a tip.
I can't understand why my fixes in 12c3bb0 commit was not working but after I have updated hw-12 branch on remote server it starts working in 9e0517b commit ? Does it mean that git pull https://mySuperSecretToken@github.com/Client-Z/hw5.git wasn't executed fine? I thoughd this command should update all my branches on remote server?

@serhiinkh
Copy link
Copy Markdown
Collaborator

Seems like there is some issues with your deploy.sh it does not pull changes.
Try to login on the server and run deploy.sh manually

@Client-Z
Copy link
Copy Markdown
Owner Author

Client-Z commented Nov 26, 2019

Seems like there is some issues with your deploy.sh it does not pull changes.
Try to login on the server and run deploy.sh manually

I found git have requested user.email field
git config --global user.email "myEmail", and now it works, thanks

@novokhatskyioleksii novokhatskyioleksii removed their request for review June 10, 2020 18:22
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