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

Keep indentation for multiline statements #992

Closed
gsemet opened this issue Jun 14, 2019 · 2 comments · Fixed by #2976
Closed

Keep indentation for multiline statements #992

gsemet opened this issue Jun 14, 2019 · 2 comments · Fixed by #2976
Labels

Comments

@gsemet
Copy link

gsemet commented Jun 14, 2019

Here an exemple of Dockerfile part the way I like to write:

RUN     true \
    &&  apt-get --quiet update --yes \
    &&  apt-get --quiet install --yes --no-install-recommends \
            curl \
            dos2unix \
            git \
            htop \
            iftop \
            iotop \
            jq \
            make \
            netcat-openbsd \
            python3-pip \
            vim \
    &&  true

I find it really efficient, merge-friendly, extendable, etc. So I want to keep it this way.

If I enable autoformatting and this extension, on save these lines are automatically rewriten to:

 echo "Acquire::https::Proxy \"http://nexus.tls.renault.fr:911\"\;" >> /etc/apt/apt.conf \

RUN     true \
    &&  apt-get --quiet update --yes \
    &&  apt-get --quiet install --yes --no-install-recommends \
    curl \
    dos2unix \
    git \
    htop \
    iftop \
    iotop \
    jq \
    make \
    netcat-openbsd \
    python3-pip \
    vim \
    &&  true
@rcjsuen
Copy link
Contributor

rcjsuen commented Jun 14, 2019

@gsemet Thank you for taking the time to open this request. I have opened rcjsuen/dockerfile-utils#62 to track this.

Have a nice day!

@gsemet
Copy link
Author

gsemet commented Jun 14, 2019

thanks!

@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants