Skip to content

Conversation

@8bcb
Copy link

@8bcb 8bcb commented Jun 13, 2021

No description provided.

@8bcb 8bcb temporarily deployed to coders-crew-heroku-depl-bskll9 June 13, 2021 12:18 Inactive
@8bcb 8bcb temporarily deployed to coders-crew-heroku-depl-iwysko June 13, 2021 12:24 Inactive
@8bcb 8bcb temporarily deployed to coders-crew-heroku-depl-oy5tmk June 13, 2021 12:46 Inactive
kamildzierzak
kamildzierzak previously approved these changes Jun 13, 2021
"storybook": "start-storybook -p 6006",
"build-storybook": "build-storybook"
},
"engines": {
Copy link
Member

Choose a reason for hiding this comment

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

Do we have to remove it because otherwise deploy on heroku doesnt work, or is there another reason? 🤔😁

Copy link
Author

Choose a reason for hiding this comment

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

I wanted to ask you if thats really necessary cause heroku cannot execute 'npm please-use-yarn' command. I assume its kind of a fake thing? If we need it i will try to find another solution but it was quite hard to find any... Anyway heroku use yarn during the deployment because it detects the yarn lock file

Copy link
Member

@kamildzierzak kamildzierzak Jun 13, 2021

Choose a reason for hiding this comment

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

For me, we can delete it if it casues problems on the heroku. Maybe I will find another way to force use yarn instead of npm, or we just have to remember that we are using yarn 😅

Copy link
Member

Choose a reason for hiding this comment

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

Heroku can use yarn instead of npm, so we can preserve those lines. As I remember the only thing you need to do is to replace npm run start:production with yarn start:production

Copy link
Author

Choose a reason for hiding this comment

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

@KonradSzwarc Yeah, you're right, now it works just fine :)

Copy link
Author

Choose a reason for hiding this comment

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

Oh, no, I said it too fast, it failed :P

"storybook": "start-storybook -p 6006",
"build-storybook": "build-storybook"
},
"engines": {
Copy link
Member

Choose a reason for hiding this comment

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

Heroku can use yarn instead of npm, so we can preserve those lines. As I remember the only thing you need to do is to replace npm run start:production with yarn start:production

package.json Outdated
"name": "coderscrew-app",
"version": "1.0.0",
"scripts": {
"predev": "docker-compose up -d",
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure if we should have this command.

  1. What if someone terminates the dev process and runs it again. Wouldn't Docker try to start the second time?
  2. It's kind of asymmetric because Docker starts automatically but needs to be terminated manually.
    I think it could be better if everyone needs to run Docker explicitly from the command line.

@8bcb 8bcb temporarily deployed to coders-crew-heroku-depl-oy5tmk June 13, 2021 20:10 Inactive
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.

4 participants