search

flyingcircusio / appenv

Self-contained bootstrapping/updating of Python applications deployed through shared repositories
BSD 3-Clause "New" or "Revised" License
5 stars 1 forks source link

Fix wording. #21

Closed icemac closed 4 years ago

icemac commented 4 years ago

As this file has to be commited to repositories owned by customers to use batou the used language might not be appropriate or even forbidden by company policies.

After merging this PR please update the version used in batou as well.

Thanks in advance.

ctheune commented 4 years ago

Well, sure. I'm not too much of a fan of cleaning up language like this, but I guess as this ends up as a copy in a customer repo that's kinda understandable. Not sure why I should change the version in my batou repo, though.

icemac commented 4 years ago

In https://github.com/flyingcircusio/batou/pull/71 I am propsing to use https://raw.githubusercontent.com/flyingcircusio/batou/master/appenv.py wich seems to be a copy of the appenv.py from here. So either there should be another URL in that PR or the changed version from here needs to be copied over there. (Sorry I did not see any documentation for batou 2 on appenv so I am just guessing here.)