[GitHub] [trafficcontrol] zrhoffman opened a new issue #6102: GitHub and IDEs do not highlight the Python Postinstall script as Python

[GitBox <gi...@apache.org>]

zrhoffman opened a new issue #6102:
URL: https://github.com/apache/trafficcontrol/issues/6102


   <!--
   ************ STOP!! ************
   If this issue identifies a security vulnerability, DO NOT submit it! Instead, contact
   the Apache Traffic Control Security Team at security@trafficcontrol.apache.org and follow the
   guidelines at https://apache.org/security regarding vulnerability disclosure.
   
   - For *SUPPORT QUESTIONS*, use the #traffic-control channel on the ASF slack (https://s.apache.org/slack-invite)
   or the Traffic Control Users mailing list (send an email to users-subscribe@trafficcontrol.apache.org to subscribe).
   - Before submitting, please **SEARCH GITHUB** for a similar issue or PR
       * https://github.com/apache/trafficcontrol/issues
       * https://github.com/apache/trafficcontrol/pulls
   -->
   
   <!-- Do not submit security vulnerabilities or support requests here - see above -->
   ## This Improvement request (usability, performance, tech debt, etc.) affects these Traffic Control components:
   <!-- delete all those that don't apply -->
   - Traffic Ops
   
   ## Current behavior:
   <!-- Describe how the current features are insufficient. -->
   The [Postintall Python script](https://github.com/apache/trafficcontrol/blob/871659244b/traffic_ops/install/bin/_postinstall) has a Bash shebang. While this itself is not an issue ([other projects](https://github.com/servo/servo/blob/master/mach) do this), combined with the fact that the file does not have a `.py` file extension, most Python IDEs will still use Bash syntax highlighting and linting for the file.
   
   This makes it less straightforward to maintain the Python postinstall script, because in order to use Python syntax highlighting and linting, it is necessary to copy or symlink it to a file that does have a `.py` file extension.
   
   ## New behavior:
   <!-- Describe how this change would improve Traffic Control -->
   The Postinstall Python script should have a `.py` file extension.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] ocket8888 closed issue #6102: IDEs do not syntax highlight the Python Postinstall script as Python

[GitBox <gi...@apache.org>]

ocket8888 closed issue #6102:
URL: https://github.com/apache/trafficcontrol/issues/6102


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org