#34085: Black shouldn't format non-Python files
-------------------------------------+-------------------------------------
     Reporter:  Jeff Triplett        |                    Owner:  nobody
         Type:  Bug                  |                   Status:  closed
    Component:  Core (Management     |                  Version:  4.1
  commands)                          |
     Severity:  Normal               |               Resolution:  needsinfo
     Keywords:  startproject black   |             Triage Stage:
                                     |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Carlton Gibson):

 * status:  new => closed
 * resolution:   => needsinfo


Comment:

 Hmm. This doesn't reproduce for me locally, so we're missing something...
 🤔

 {{{
 Last login: Tue Oct 11 10:24:11 on ttys000
 carlton@Carltons-MacBook-Pro ~ % mktmpenv
 ...
 (tmp-485998ddadf9b85) carlton@Carltons-MacBook-Pro tmp-485998ddadf9b85 %
 pip install Django
 ...
 Successfully installed Django-4.1.2 asgiref-3.5.2 sqlparse-0.4.3
 (tmp-485998ddadf9b85) carlton@Carltons-MacBook-Pro tmp-485998ddadf9b85 %
 django-admin startproject \
     --extension=ini,py,yml \
     --template=https://github.com/jefftriplett/django-
 startproject/archive/main.zip \
     example_project
 (tmp-485998ddadf9b85) carlton@Carltons-MacBook-Pro tmp-485998ddadf9b85 %
 cat example_project/requirements/requirements.in
 django-click
 Django<4.2
 environs[django]
 psycopg2-binary
 whitenoise

 black
 django-test-plus
 ipdb
 model-bakery
 pip-tools
 pytest
 pytest-cov
 pytest-django
 (tmp-485998ddadf9b85) carlton@Carltons-MacBook-Pro tmp-485998ddadf9b85 %
 black --version
 black, version 21.10b0
 }}}

 Do you think Oliver could dig in a bit to spot where the issue is coming
 up? 🤔

 > While I am a fan of Black, I personally think we should have some type
 of --skip-black or --skip-formatters because it's disruptive to have Black
 installed globally and have this be opt-in by default.

 This was discussed (at some length) during the development of the feature.
 It was decided against. Rather set the `PATH` explicitly if you have black
 installed globally but **do not** want to apply it for a particular
 invocation.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/34085#comment:2>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/01070183c83d4991-f6902649-8d34-4b16-abce-d9735c28e257-000000%40eu-central-1.amazonses.com.

Reply via email to