Umair Khan
f22b2d27f3
check-capitalization: Add no-generate option.
...
Adds the option to not generate translation files.
2017-07-03 10:10:43 -04:00
Umair Khan
f1fded08ab
check-frontend-i18n: Generate only English translations.
2017-07-03 10:10:43 -04:00
Umair Khan
50ee14d390
check-capitalization: Generate only English translations.
2017-07-03 10:10:43 -04:00
sinwar
3d0b5b14be
provision: Avoid spending ~1.5s on compilemessages in no-op case.
...
This brings the total no-op provision time down to around 7.3s!
Tweaked by tabbott to clean up the code, close file descriptions, and
avoid issues with a somehow corrupted file.
Fixes #5185 .
2017-07-01 11:27:54 -07:00
Harshit Bansal
a876ebb52a
build_emoji: Close open file objects after reading.
2017-07-01 11:14:24 -07:00
Harshit Bansal
8c92024fae
build_emoji: Add `codepoint_to_name` dict to emoji_codes.js.
...
This dict is a mapping from codepoint to canonical name.
2017-07-01 11:14:24 -07:00
Harshit Bansal
fee4065571
build_emoji: Generate `codepoint_to_name.json` file.
...
Store the `codepoint_to_name` map in a JSON file which can be used
by the rest of codebase(zerver) to convert emoji codepoints to
canonical names.
2017-07-01 11:14:24 -07:00
Umair Khan
f60d4849b7
Revert "lint: Don't allow handlerbars within translatable text."
...
This reverts commit 3c7b9cdc94
.
2017-06-30 10:12:31 -04:00
Umair Khan
74d83cc47e
lint: Disallow handlebars within {{#tr}}{{/tr}}.
...
This commit adds a new linter which runs from tools/travis/backend.
It runs over the translations.json file and checks if any of the
translatable string contains handlebars in it.
Fixes #5544
2017-06-30 10:12:31 -04:00
Steve Howell
6cf4c65d92
node tests: Add 100% coverage for channel.js.
2017-06-30 09:48:04 -04:00
Rishi Gupta
85d38bd17b
emails: Remove DEFAULT_FROM_EMAIL from prod_settings_template.
2017-06-29 17:54:33 -07:00
Umair Khan
3c7b9cdc94
lint: Don't allow handlerbars within translatable text.
...
Fixes : #5544
2017-06-29 07:39:20 -04:00
Steve Howell
198c363b89
Fix errors in create-test-api-docs.
...
Our recent Django upgrade removed some info from the file
used by this tool, but there is still useful info, so I
resurrected the tool.
2017-06-27 08:56:39 -04:00
Yago González
33e4855306
lint: Ban non-unique operationIds in Swagger.
...
The Swagger specification indicates that all operationId values should be
unique. However, SwaggerParser doesn't complain during validation if that
doesn't happen, so this commit adds our own method to identify these
cases.
Also, the violations of this rule have been fixed.
2017-06-26 22:02:26 -04:00
Yago González
00a7f89ba9
node tests: Add coverage to composebox_typeahead.js.
2017-06-26 19:33:25 -04:00
Yago González
7e3154f7bc
lint: Ignore tildes in mustache tag template checker.
...
Handlebars allows putting tildes (~) by the braces in mustache tags to
strip whitespaces on that side of the tag.
{{#if foo~}}
<p>Bar</p>
{{~/if}}
This way, the linter ignores any potential tilde at the ends of Handlebars
tags so we can use this feature without lint fails.
2017-06-26 19:33:25 -04:00
Aditya Bansal
1cb5c9faa8
node tests: Enforce 100% test coverage upon emoji.js.
2017-06-26 14:24:20 -04:00
Aditya Bansal
d1fe75ce3b
node tests: Enforce 100% node coverage in activity.js.
2017-06-26 14:24:20 -04:00
Eeshan Garg
4ad951f256
integrations/perforce: Migrate docs to Markdown.
2017-06-26 12:30:04 -04:00
Tommy Ip
f521b03474
lint: Check if commit message is not empty before running gitlint.
2017-06-25 11:29:14 -04:00
Joshua Pan
a63fd95b6a
test-js-with-node: Add check for fully covered files that are not enforced.
2017-06-23 16:05:16 -04:00
Joshua Pan
c625a80b90
test-js-with-node: Extract check_line_coverage function.
2017-06-23 16:05:16 -04:00
Joshua Pan
3557c47d76
node tests: Enforce 100% node coverage in user_events.js.
2017-06-23 16:05:16 -04:00
Aditya Bansal
eb39e82629
Update enforce_fully_covered list with pm_list.js.
2017-06-23 11:13:47 -04:00
Aditya Bansal
18eeb962fe
setup-static-analysis: Add wrapper to retry dependancy installation.
2017-06-23 08:23:45 -04:00
Aditya Bansal
a20249b126
tooling: Add tool to clear unused emoji cache.
...
In this commit, we are adding a new tool which can be used to
clear out the emoji cache. Cache cleanup is a good thing to in
general and as well helps keep travis cache small (meaning faster
builds).
Credits to @HarshitOnGitHub as well for suggesting the use of
emoji symlink.
2017-06-22 19:17:12 -04:00
Harshit Bansal
0b03fd97a2
webpack: Don't include katex bundle in base.html.
...
Since we use katex only in the main webapp, we don't need it in things
like landing page etc, so it is better to move it to index.html.
2017-06-22 19:06:32 -04:00
Harshit Bansal
99b12ed0ca
webpack: Use `common.js` to inject webpack dependencies in dev.
2017-06-22 19:06:32 -04:00
Harshit Bansal
c0e4f24a25
webpack: Migrate `common.js` bundle to webpack.
2017-06-22 19:06:32 -04:00
sinwar
0e24930770
provision: Avoid spending ~3s on apt operations in no-op provision.
...
Fixes #5186
2017-06-22 09:37:40 -04:00
Vaida Plankyte
0ed4e22e54
docs: Fix lint/whitespace errors in GCI docs.
2017-06-22 09:30:28 -04:00
Robert Hönig
d45d9c9ab5
bots: Add linter rule to prevent open() calls.
...
Addresses #5395 .
2017-06-21 10:16:59 -04:00
Robert Hönig
fd41a481e4
linter: Fix `include_only` rule path comparison.
...
In order for the `include_only` linter rule to not have
any side effects, we need to explicitly add a trailing '/'
after every directory we want to include.
2017-06-21 10:16:59 -04:00
Steve Howell
e49afe3ebc
lint: Add code to suggest removing exclusions.
...
If excluded lines no longer exist in a file, print a message
asking the user to remove the exclusion.
2017-06-21 08:12:58 -04:00
Joshua Pan
bd9459f27d
node test: Enforce 100% node coverage in typeahead_helper.js.
2017-06-21 00:03:05 -04:00
Steve Howell
572ea370ea
Speed up custom checkers in `./tools/lint`.
...
The main thing here is to make looping over lines be the inner
loop, instead of looping over rules. This keeps regexes in
cache, and it also avoids some O(N) checks.
This is a significant speedup for me, reducing time from 16s
to 11s.
2017-06-20 22:26:12 -04:00
Harshit Bansal
c50b3672c2
webpack: Migrate `signup.js` bundle to webpack static asset pipeline.
2017-06-20 09:45:26 -04:00
Harshit Bansal
3953734a1c
webpack: Migrate `stats.js` bundle to webpack static asset pipeline.
2017-06-20 09:45:26 -04:00
Umair Khan
bccd2caacb
testing: Fix search key.
...
The problem this commit solves is related to how we search for
testcases in the test files. We use a simple 'search_key' in file_data.
This will return a false positive if there is a 'longer_search_key' in
file_data.
While searching for TestCases we should use a longer key to remove the
possibility of collision. Using 'class TestCase(' should be precise
enough.
Fixes #4983
2017-06-20 08:35:07 -04:00
Cynthia Lin
9777496046
repository: Update information with Codecov migration.
2017-06-20 07:17:39 -04:00
Aditya Bansal
b19134d4c2
travis: Remove docker-engine from held package list in production.
...
We are doing this in order to avoid facing troubles when travis
updates its trusty images. Here is a link to the blog post.
https://blog.travis-ci.com/2017-06-19-trusty-updates-2017-Q2
2017-06-20 06:45:40 -04:00
Cory Lynch
005d06eb38
message_store.js: Add tests for message id change.
...
This gets message_store.js to 100% line coverage.
2017-06-20 06:24:27 -04:00
Aditya Bansal
7980462010
setup-backend: Add wrapper to retry provisioning in case it fails.
...
In this commit we add a wrapper around the provisioning inside
setup-backend to trigger another try at provisioning if it fails
in hopes of it will run this time fine if it was interuppted by a
network issue the first time.
Fixes : #2034 .
2017-06-19 16:14:11 -04:00
Aditya Bansal
924d8266c0
setup-production: Add a wrapper to retry provisioning when it fails.
...
In this commit we basically add a wrapper around provisioning
inside setup-production to trigger a retry in case of failure.
2017-06-19 16:14:11 -04:00
Aditya Bansal
fab4a30cce
production-helper: Add wrapper to retry apt-get dist-upgrade.
...
In this commit we add a conditional inside production helper to just
re run apt-get dist-upgrade in case it fails the first time.
2017-06-19 16:14:11 -04:00
vaibhav
33c0c00cd6
Outgoing webhook System: first Iteration of outgoing webhook UI.
2017-06-19 16:05:54 -04:00
Joshua Pan
23dc32a0f3
node tests: Enforce 100% coverage in markdown.js and presence.js.
2017-06-19 08:00:55 -04:00
Joshua Pan
d1408125b2
tools: Make sure missing lines in node test coverage are correct.
2017-06-19 07:59:50 -04:00
Aditya Bansal
7de4c340f6
tooling: Add tool to clean unused NPM cache.
2017-06-19 07:55:23 -04:00
Tommy Ip
e9fd41bb66
lint: Check if the first word of a commit message is imperative.
...
Only check the first word using a black list approach to reduce
force positive as shown in the current implementation.
2017-06-19 07:01:52 -04:00