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
Updates #18
Updates #18
Conversation
This updates the versions in the requirements file, removes unneeded imports in test files and makes a small change so we don't need test-utils anymore.
Moving these to the base makes it way easier to configure tower. For example, instead of this: DOMAIN_METHODS = { 'messages': [ ('fjord/**.py', 'tower.management.commands.extract.extract_tower_python'), ] } we can do this: DOMAIN_METHODS = { 'messages': [ ('fjord/**.py', 'tower.extract_tower_python'), ] } I added an import to tower.management.commands.extract so that older tower configurations continue to work.
This documents tower. There's probably some things missing, but it'll make people (aka me) much happier.
I just noticed there are two pull requests that have been around for many LDAP password resets. Please don't let this one languish in pull request purgatory forever. It doesn't change any functionality. It's backwards compatible with existing projects. Help me tower maintainers. You are my only hope. |
* Jinja2 and Jingo | ||
* translate-toolkit | ||
|
||
See requirements.txt for details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could have linked to it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh... you're right. I'll do another pr for that.
beat me to it! Thanks |
Thank you both! I really appreciate the promptness. Also, hi @davedash! I miss you! |
This fixes a couple of basic problems I have with tower:
r?