<p>In <a href="https://github.com/opentechinstitute/mlab-observatory/pull/69#discussion_r28548156">environment_bootstrap.py</a>:</p>
<pre style='color:#555'>> @@ -24,6 +24,11 @@
>  
>  def setup_environment(environment_type):
>    link_name = 'static/js/observatory/js/paths.js'
> +
> +  if os.path.exists(link_name):
> +    print 'Warning: Replacing existing file: %s' % link_name
> +    os.remove(link_name)
</pre>
<p>This is worrying. "I'm screwing up your system riiiiiiiiight NOW."</p>

<p>Don't give a warning after you perform an irreversible action.  Either add some complication so that people have to affirmatively confirm that they want you to delete and existing file, or change the text to "Warning: replaced existing file".</p>

<p>I think the best thing of all would be only to remove the link if os.path.islink(link_name) returned true, and to error out otherwise.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/opentechinstitute/mlab-observatory/pull/69/files#r28548156">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AC7qBIwL__M-kohtAGJo3VARtX44JrRmks5oABcCgaJpZM4ECO9S.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
  <div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
    <link itemprop="url" href="https://github.com/opentechinstitute/mlab-observatory/pull/69/files#r28548156"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>