Skip to content

feat: build and run from Docker Support#241

Open
rajpratik71 wants to merge 2 commits intoyeoman:mainfrom
yeoman-projects:feat/docker-support
Open

feat: build and run from Docker Support#241
rajpratik71 wants to merge 2 commits intoyeoman:mainfrom
yeoman-projects:feat/docker-support

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 rajpratik71@gmail.com

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 <rajpratik71@gmail.com>
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

Co-authored-by: Ulises Gascón <UlisesGascon@users.noreply.github.com>
@UlisesGascon UlisesGascon added this to the 6.0.0 milestone Mar 23, 2023
@JoshuaKGoldberg
Copy link
Contributor

👋 Just following up @UlisesGascon, can we merge & release this? Are there any blockers? Seems like a non-breaking change to me?

@UlisesGascon
Copy link
Member

I will like to make some additional work, like update the base image, node version... and include an step in the CI to check that this is working. I will open a new PR probably with all the changes

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

Labels

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants