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

feat: build and run from Docker Support #241

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Rajpratik71
Copy link

Build and run the generator from docker image support added.

The main aim and motivation behind doing this is, because
of a no. of different generator are there,
which are compatible and works with the different version of generator
and also might be incompatible with other generator or versions of generator

Signed-off-by: Pratik Raj [email protected]

Build and run the generator from docker image support added.

The main aim and motivation behind doing this is, because
of a no. of different generator are there,
which are compatible and works with the different version of generator
and also might be incompatible with other generator or versions of generator

Signed-off-by: Pratik Raj <[email protected]>
Base automatically changed from master to main January 24, 2021 06:32
echo 'yeoman:yeoman' |chpasswd && \
mkdir /home/yeoman/generator && \
export DEBIAN_FRONTEND=noninteractive && \
export TZ=Europe\Paris && \
Copy link
Member

Choose a reason for hiding this comment

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

Why the TZ should be Paris? Do we require to have TZ specifically defined ?

I believe that the user can always provide that as a env argument:

docker run -rm -e TZ=Europe/Madrid -it myimage

docker/Dockerfile Outdated Show resolved Hide resolved
@UlisesGascon UlisesGascon added this to the 6.0.0 milestone Mar 23, 2023
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