0ddad98c98
features.html: Cleanup to use 4 space and consistant indentation.
2017-10-25 10:58:57 -07:00
3fb0f7c092
home.html: Cleanup to use 4 space indentation.
...
Tweaked by tabbott to also make the translation tags make more sense.
2017-10-25 10:58:49 -07:00
618ace1f5c
build-release-tarball: Fix missing CSS-inlined email templates.
...
These apparently we're being included.
2017-10-20 14:08:25 -07:00
c9e29b1c54
check-templates: Document status of remaining bad_files items.
2017-10-19 16:56:46 -07:00
2fb46df005
check-templates: Remove exclude rules for removed HTML files.
...
These webhook integrations were converted to using markdown some time
ago.
2017-10-19 16:56:46 -07:00
fc4b87ed04
templates: Clean some whitespace issues in markdown_help.
...
Unfortunately, the rest cannot be easily fixed.
2017-10-19 16:56:46 -07:00
e80fc93e87
templates: Fix search_operators indentation.
2017-10-19 16:56:46 -07:00
59344d77df
templates: Clean various items off HTML linter bad list.
...
Most of these already pass the linter; for the ones that don't, the
fixes are small and don't make the HTML any worse.
2017-10-19 16:56:00 -07:00
540cae19a8
puppet: Remove obsolete sparkle configuration.
...
Sparkle was the auto-update system used by the legacy desktop app. We
haven't been capable of using it for auto-update in years, so there's
no reason to keep around the configuration.
The new Electron app uses a different system anyway.
2017-10-19 16:35:55 -07:00
4ddb812354
check_subdomain: Remove case for user_subdomain being None.
...
With REALMS_HAVE_SUBDOMAINS always on, user_subdomain is never None,
so this was just dead code.
2017-10-18 22:35:07 -07:00
1ab2ca5986
subdomains: Extract zerver.lib.subdomains library.
...
These never really belonged with the rest of zerver.lib.utils.py, and
having a separate library makes it easier to enforce full test
coverage.
2017-10-18 22:27:48 -07:00
c8090aafa4
lint: Remove several unused imports.
2017-10-18 10:56:11 -07:00
fe3a8084d8
mypy: Fix annotations for linter.
2017-10-18 10:50:25 -07:00
2bc14d256f
lint: Ban two spaces after comma in JS code.
...
We exclude the frontend tests, mostly because the lint rule isn't that
precise, and the test code has some sample user input that's a bit
funny.
2017-10-18 10:22:18 -07:00
8afe9835f6
lint: Ban two spaces after comma in prose.
2017-10-18 10:12:41 -07:00
b634d9f38b
lint: Ban two spaces after comma in CSS.
2017-10-18 10:11:33 -07:00
194e102077
linter: Add rule against multiple whitespaces after comma.
2017-10-18 10:04:23 -07:00
691598a88b
py3: Remove "from six.moves import range".
...
This is no longer required, since in Python 3, this is what the range
built-in does.
2017-10-17 23:28:14 -07:00
a603a4f9f5
Remove `from __future__ import absolute_import`.
...
Except in:
- docs/writing-bots-guide.md, because bots are supposed to be Python 2
compatible
- puppet/zulip_ops/files/zulip-ec2-configure-interfaces, because this
script is still on python2.7
- tools/lint
- tools/linter_lib
- tools/lister.py
For the latter two, because they might be yanked away to a separate repo
for general use with other FLOSS projects.
2017-10-17 22:59:42 -07:00
bcca964548
test-js-with-node: Fix doing too many imports before sanity_check.
...
This wasn't benefitting from the nice error message.
2017-10-17 22:22:00 -07:00
9c5e0971f4
sanity_check: Check a few packages.
...
This just makes our check for whether we're actually in a Zulip
development environment better.
2017-10-17 22:21:33 -07:00
508686bed4
provision: Fix buggy bash_profile management code.
...
Apparently, the previous version would just append to .bash_profile
every single time :(.
2017-10-17 21:22:32 -07:00
3fed995a41
provision_version: Move to under UUID_VAR_PATH.
2017-10-17 21:15:58 -07:00
3983ebce9a
test_fixtures: Move test_db_status state under UUID_VAR_PATH.
...
This should mean that maintaining two Zulip development environments
using the same Git checkout no longer has caching problems keeping
track of the migration status.
2017-10-17 21:15:58 -07:00
b7c3b4df0c
migration_status: Switch to passing the path in Python.
...
This is an important prerequisite to changing these paths used in
provisioning.
2017-10-17 21:15:58 -07:00
a7c639cff7
provision: Use UUID in path to compilemessages hash.
2017-10-17 21:15:58 -07:00
2ae2a94444
provision: Stop using shared var/ for caching apt state.
...
This didn't work at all when one did a `vagrant destroy` and then
`vagrant up`, because the cache state would be preserved even though
the machine is gone.
Fixes #5981 .
2017-10-17 21:15:58 -07:00
3e258d11e0
Travis: Use travis.yml built-in way to install moreutils.
2017-10-17 09:59:59 -07:00
e1660462e1
linter: Add custom shebang rules section.
...
Tweaked by tabbott for clarity and to use print_err.
2017-10-12 11:21:38 -07:00
6176d0fbca
json: Replace most use of simplejson with json.
...
This is progress towards removing simplejson as a dependency.
2017-10-11 22:55:35 -07:00
f8b103bd98
tools: Add a test for checking locked requirements.
...
This commit adds a test to check if the user forgot to run
`tools/update-locked-requirements` after updating dependencies.
Modified by tabbott to disable it by default, since it takes over a
minute to run.
Fixes : #6324 .
2017-10-11 22:55:15 -07:00
0391619f48
tools: Modify `update-locked-requirements` to take optional arguments.
...
This commit modifies the `update-locked-requirements` tool to take
output file path as optional arguments.
2017-10-11 22:44:54 -07:00
bfaf0fb3b2
build-release-tarball: Stop using .gitattributes.
...
Unfortunately, GitHub's web UI for generating release tarballs uses
`.gitattributes` to control what files to download, and thus if you
downloaded a source tarball for older Zulip versions using the GitHub
web UI, you'd be missing important files.
We fix this for future releases by moving the blacklist out of
.gitattributes.
Fixes #129 .
2017-10-11 21:51:42 -07:00
80e50f4b57
test-backend: Add goal of full coverage on zerver/*.py.
...
Like the zerver/lib changes, this should help make the backend
coverage TODO list a bit more explicit.
2017-10-11 20:17:37 -07:00
ce579d5896
test-backend: Make test coverage gaps in zerver/lib more explicit.
...
This will make it much less likely that we'll have more regressions of
files losing 100% test coverage like push_notifications.py had
recently.
2017-10-11 20:17:37 -07:00
676a6af8cb
push_notifications: Get back to 100% test coverage.
...
We didn't have tests for a few new code paths.
2017-10-11 20:17:27 -07:00
f6834bb60e
tools: Remove `.pl` extension from user script.
2017-10-11 12:52:36 -07:00
d556f677b2
tools: Consistently use hyphen in user script names.
2017-10-11 12:52:36 -07:00
61aebd036f
tools: Remove `.py` extensions from user scripts.
2017-10-11 12:52:36 -07:00
df948c7ea5
Remove useless shebang lines.
2017-10-11 12:52:36 -07:00
c04ff1cc79
mypy: Any->Call in create_single_page() in create-test-api-docs.
2017-10-11 12:16:50 -07:00
baee129eda
lint: Exclude style= rules for email log.
...
This is temporary, because this PR was written before these rules were
put in place.
2017-10-11 10:41:40 -07:00
34bafa654c
mypy: Fix future syntax errors and other minor mistakes.
...
When we move to the Python 3 mypy syntax, we can't reference a class
before its definition.
2017-10-10 22:27:16 -07:00
416f34c80e
setup-production: Try to fix Travis failures by blocking mongodb repo.
...
It appears the mongodb repo is not accessible by Travis CI right now.
This is sadly our problem, because Travis puts a bunch of crap in
their apt `sources.list` file, so `apt-get update` starts failing.
2017-10-08 20:06:42 -07:00
810532641e
activity.js: Extract get_filtered_and_sorted_user_ids().
...
This function was extracted from build_user_sidebar(). We
also slightly streamlined it to not unnecessarily call
filter() when the filter text was blank. This extraction
also eliminated the need for us to have the two-line
filter_and_sort() function.
Also, we get to 100% coverage in this commit.
2017-10-08 12:01:59 -07:00
86d372f9ce
lint: Prevent "userid" in code.
...
We should always use user_id for consistency sake.
2017-10-07 12:16:45 -07:00
9cf44a77da
lint: Narrow the space-around-% patterns, remove exclusions.
...
This lint rule has bitten me a couple of times in working on logging.
These regex rules will inevitably be heuristic, but we can make it a bit more
specific so that the heuristic mainly means it could occasionally miss
something, rather than get in the way with an obviously wrong complaint.
2017-10-06 19:21:40 -07:00
a9a22a3651
lint: Use new exclude_pattern system to clean up inline style lint.
2017-10-06 13:16:32 -07:00
7ff697d053
linter: Add rule against using `pk` instead of `id`.
...
There's one exception here, for model._meta.pk. To support this
nicely, we added this exclude-pattern feature.
2017-10-06 12:56:26 -07:00
64b8930225
linter: Add rules against improper shebangs.
2017-10-06 12:46:19 -07:00
Aditya Bansal
Tim Abbott
rht
derAnfaenger
Harshit Bansal
neiljp (Neil Pilgrim)
Steve Howell
Greg Price