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.

Sign up for GitHub

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

Jump to bottom

#79 - feat: dockerize #86

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

#79 - feat: dockerize #86

wants to merge 3 commits into from

Conversation

km4
Copy link

@km4 km4 commented Jan 31, 2023

No description provided.

@RacyMind RacyMind self-requested a review February 3, 2023 17:45
@dithh
Copy link
Contributor

dithh commented Feb 4, 2023

super, ze jest docker.
ale czy frontowych appkach czas potrzebny na skonfigurwoanie dockera przynosi faktycznie wymierne korzysci?
mamy nvm, wiec zmiana wersji node to jedna linijka w terminalu.
jakie jeszcze sa zalety?
pytam z czystej ciekawosci, to nie hejt :D

@km4
Copy link
Author

km4 commented Feb 5, 2023

Nie wiem jaki zamysł miał twórca issue. W tak małym projekcie jest mały gain jeśli chodzi o dokeryzacje (w developmencie). Jak dobre zauwazyles nvm załatwia wiekszosc spraw, jednak nie każdy wie że coś takiego istnieje.

@lsochaSopamo lsochaSopamo self-requested a review February 6, 2023 13:35
@lsochaSopamo
Copy link
Collaborator

Nie wiem jaki zamysł miał twórca issue. W tak małym projekcie jest mały gain jeśli chodzi o dokeryzacje (w developmencie). Jak dobre zauwazyles nvm załatwia wiekszosc spraw, jednak nie każdy wie że coś takiego istnieje.

Na grupie programistycznej, gdzie wrzuciłem info o kalkulatorze, kilka osób zasugerowało dodanie Dockera, by nie było problemu z różnymi wersjami node. Dla mnie także nvm jest wystarczający. Zdecydowałem się dodać issue z Dockerem, ponieważ dodanie Dockera nie jest jakoś skomplikowane dla kalkulatora, a dla części programistów może być przydatny. Przy projekcie open source trzeba zapewnić wsparcie programistom z różnym podejściem.

lsochaSopamo
lsochaSopamo requested changes Feb 6, 2023
View reviewed changes
Dockerfile Show resolved Hide resolved
@km4 km4 requested review from lsochaSopamo and removed request for RacyMind February 7, 2023 20:23
RacyMind
RacyMind requested changes Feb 14, 2023
View reviewed changes
Copy link
Owner

@RacyMind RacyMind left a comment

Choose a reason for hiding this comment

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

Podczas próby uruchomienia mam następujące błędy
image

@km4 km4 closed this Feb 15, 2023
@km4 km4 reopened this Feb 15, 2023
@km4
Copy link
Author

km4 commented Feb 16, 2023

Podczas próby uruchomienia mam następujące błędy image

w takim wypadku pewnie musisz zmienic uprawnienia albo trzeba pierw utworzyc wolumen. co wolisz ? jakis tip ?

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

@RacyMind RacyMind RacyMind requested changes

@lsochaSopamo lsochaSopamo Awaiting requested review from lsochaSopamo

Requested changes must be addressed to merge this pull request.

Assignees

@km4 km4

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@km4 @dithh @lsochaSopamo @RacyMind