Remove extra new lines at the ends of Zulip authoried files.

Fixes #1627.

[tweaked by tabbott to avoid patching third-party modules, for now]

.tx/config

@@ -14,4 +14,3 @@ source_lang = en
 type = KEYVALUEJSON
 file_filter = static/locale/<lang>/translations.json
 lang_map = zh-Hans: zh-CN
-

bots/log2zulip

@@ -110,4 +110,3 @@ if __name__ == "__main__":
             os.remove(lock_path)
         except OSError as IOError:
             pass
-

bots/sync-public-streams

@@ -64,4 +64,3 @@ if __name__ == "__main__":
         f.close()
 
         subprocess.call(["mv", "/home/zulip/public_streams.tmp", "/home/zulip/public_streams"])
-

contrib_bots/lib/readme.md

@@ -106,4 +106,3 @@ If you are not interested in running your bots on the
 server, then you can still use the full Zulip API.  The
 hope, though, is that this architecture will make
 writing simple bots a quick/easy process.
-

docs/custom-apps.md

@@ -402,4 +402,3 @@ highly locked down, e.g. enforce that they truly only have read access
 to Zulip messages.
 - **UI**: We will want to provide some UI features that give admins and/or
 regular users visibility into which server-side apps are running.
-

docs/index.rst

@@ -117,4 +117,3 @@ Indices and tables
 * :ref:`genindex`
 * :ref:`modindex`
 * :ref:`search`
-

docs/integration-guide.md

@@ -582,5 +582,3 @@ request:
 
 If you would like feedback on your integration as you go, feel free to submit
 pull requests as you go, prefixing them with `[WIP]`.
-
-

docs/linters.md

@@ -236,4 +236,3 @@ linter or an automated test.
 Our goal is to have most common style issues by caught by the linters, so new
 contributors to the codebase can efficiently fix produce code with correct
 style without needing to go back-and-forth with a reviewer.
-

docs/prod-install.md

@@ -163,4 +163,3 @@ help" stream in the Zulip developers'
 chat](https://zulip.tabbott.net/#narrow/stream/installation.20help)
 for realtime help or email zulip-help@googlegroups.com with the
 traceback and we'll try to help you out!
-

docs/testing-with-casper.md

@@ -121,4 +121,3 @@ for writing Casper tests in addition to the debugging notes below:
     between two `casper.then` blocks actually runs before either of
     them. See this for more details about how Casper works:
     <http://docs.casperjs.org/en/latest/faq.html#how-does-then-and-the-step-stack-work>
-

docs/testing-with-django.md

@@ -242,4 +242,3 @@ modules.  You can use the `--coverage` option to generate coverage
 reports, and new code should have 100% coverage, which generally requires
 testing not only the "happy path" but also error handling code and
 edge cases.
-

docs/testing-with-node.md

@@ -129,4 +129,3 @@ here is this:
 >     global.narrow.stream = function () {
 >         return 'office';
 >     };
-

docs/writing-views.md

@@ -362,5 +362,3 @@ def api_pagerduty_webhook(request, user_profile, client,
 ```
 The `client` will be the result of `get_client("ZulipPagerDutyWebhook")`
 in this example.
-
-

frontend_tests/node_tests/alert_words.js

@@ -76,4 +76,3 @@ var alert_domain_message = { sender_email: 'another@zulip.com', content: '<p>now
     alert_words.process_message(alert_domain_message);
     assert.equal(alert_domain_message.content, '<p>now with link <a href="http://www.alerttwo.us/foo/bar" target="_blank" title="http://www.alerttwo.us/foo/bar">www.<span class=\'alert-word\'>alerttwo</span>.us/foo/bar</a></p>');
 }());
-

frontend_tests/node_tests/dict.js

@@ -171,5 +171,3 @@ set_global('blueslip', {});
     d.del('foo');
     assert.equal(d.num_items(), 1);
 }());
-
-

frontend_tests/node_tests/dispatch.js

@@ -721,5 +721,3 @@ run(function (override, capture, args) {
     assert_same(args.message_id, 99);
     assert_same(args.new_value, true); // for 'add'
 });
-
-

frontend_tests/node_tests/filter.js

@@ -496,4 +496,3 @@ function get_predicate(operators) {
     assert.equal(Filter.describe(narrow), string);
 
 }());
-

frontend_tests/node_tests/unread.js

@@ -387,4 +387,3 @@ var zero_counts = {
     assert(unread.message_unread({flags: []}));
     assert(!unread.message_unread({flags: ['read']}));
 }());
-

frontend_tests/zjsunit/finder.js

@@ -51,4 +51,3 @@ exports.find_files_to_run = function () {
 return exports;
 }());
 module.exports = finder;
-

frontend_tests/zjsunit/namespace.js

@@ -54,4 +54,3 @@ exports.stub_out_jquery = function () {
 return exports;
 }());
 module.exports = namespace;
-

frontend_tests/zjsunit/output.js

@@ -120,4 +120,3 @@ exports.append_test_output = function (output) {
 return exports;
 }());
 module.exports = output;
-

frontend_tests/zjsunit/render.js

@@ -39,4 +39,3 @@ exports.use_template = function (name) {
 return exports;
 }());
 module.exports = render;
-

puppet/zulip/manifests/camo.pp

@@ -16,5 +16,3 @@ class zulip::camo {
     content => template("zulip/camo_defaults.template.erb"),
   }
 }
-
-

puppet/zulip/manifests/nginx.pp

@@ -50,4 +50,3 @@ class zulip::nginx {
     ensure     => running,
   }
 }
-

puppet/zulip/templates/postgresql/9.3/postgresql.conf.template.erb

@@ -574,4 +574,3 @@ log_autovacuum_min_duration = 100
 autovacuum_freeze_max_age = 2000000000
 vacuum_freeze_min_age     = 1000000000
 vacuum_freeze_table_age   = 1800000000
-

puppet/zulip_internal/files/apache/ports.conf

@@ -24,4 +24,3 @@ Listen 80
 <IfModule mod_gnutls.c>
     Listen 443
 </IfModule>
-

puppet/zulip_internal/files/mediawiki/Auth_remoteuser.php

@@ -429,4 +429,3 @@ class Auth_remoteuser extends AuthPlugin {
 		return ucfirst( $username );
 	}
 }
-

puppet/zulip_internal/files/sparkle/mac/sparkle.xml

@@ -84,4 +84,4 @@
          <enclosure length="8700959" sparkle:dsaSignature="MC0CFQCqn2WvVmBo54A6TjiuPxk2nFemYgIUW/So38cwNogoo86zpuS/ql3rST4=" sparkle:version="0.3.4" type="application/octet-stream" url="https://zulip.com/dist/apps/mac/Zulip-0.3.4.tar.bz2"/>
       </item>
    </channel>
-</rss>
+</rss>

puppet/zulip_internal/files/sparkle/sso/mac/sparkle.xml

@@ -47,4 +47,4 @@
          <enclosure length="8835310" sparkle:dsaSignature="MCwCFH+m2i53YLv33BAElEFBiGE7OZCwAhRhM42psyPDsjV8oVAthjE70en8nQ==" sparkle:version="0.3.11" type="application/octet-stream" url="https://zulip.com/dist/apps/sso/mac/Zulip-0.3.11.tar.bz2"/>
       </item>
    </channel>
-</rss>
+</rss>

puppet/zulip_internal/files/sparkle/sso/win/sparkle.xml

@@ -34,4 +34,4 @@
          <enclosure sparkle:version="0.3.11" type="application/octet-stream" url="https://zulip.com/dist/apps/sso/win/zulip-0.3.11.exe"/>
       </item>
    </channel>
-</rss>
+</rss>

puppet/zulip_internal/files/sparkle/win/sparkle.xml

@@ -57,4 +57,4 @@
          <enclosure sparkle:version="0.3.4" type="application/octet-stream" url="https://zulip.com/dist/apps/win/zulip-0.3.4.exe"/>
       </item>
    </channel>
-</rss>
+</rss>

puppet/zulip_internal/files/trac.ini

@@ -218,4 +218,3 @@ max_size = 262144
 render_unsafe_content = false
 safe_schemes = cvs, file, ftp, git, irc, http, https, news, sftp, smb, ssh, svn, svn+ssh
 split_page_names = false
-

scripts/nagios/cron_file_helper.py

@@ -35,4 +35,3 @@ def nagios_from_file(results_file):
             data = pieces[3]
 
     return (ret, "%s: %s" % (state, data))
-

static/js/muting_ui.js

@@ -121,4 +121,3 @@ $(function () {
 
 return exports;
 }());
-

static/js/resize.js

@@ -259,4 +259,3 @@ exports.handler = function (e) {
 
 return exports;
 }());
-

static/js/setup.js

@@ -42,5 +42,3 @@ $(function () {
     }
 
 });
-
-

static/styles/activity.css

@@ -48,4 +48,3 @@ tr.admin td:first-child {
     font-weight: bold;
     color: #c00;
 }
-

static/styles/fonts.css

@@ -141,4 +141,3 @@
     font-style: italic;
 
 }
-

static/styles/settings.css

@@ -471,4 +471,3 @@ input[type=checkbox].inline-block {
     word-break: break-all;
     white-space: normal;
 }
-

static/templates/invite_subscription.handlebars

@@ -10,4 +10,3 @@
   </label>
   {{/each}}
 </div>
-

templates/confirmation/mituser_confirmation_email_body.txt

@@ -19,4 +19,3 @@ problems.
 Cheers,
 
 All of us here at Zulip
-

templates/tests/test_markdown.html

@@ -3,4 +3,3 @@ header
 {{ markdown_test_file|render_markdown_path }}
 
 footer
-

tools/deprecated/stress-test/.gitignore

@@ -1,2 +1 @@
 node_modules
-

tools/do-destroy-rebuild-database

@@ -19,4 +19,3 @@ python manage.py populate_db -n100 --threads=1
 if [ -e ~/.zuliprc ]; then
     python manage.py sync_api_key
 fi
-

tools/do-destroy-rebuild-test-database

@@ -4,4 +4,3 @@ set -x
 
 # This is a really simple wrapper script, pretty much for documenting clarity
 "`dirname "$0"`/../tools/setup/generate-fixtures" --force
-

tools/fetch-rebase-pull-request

@@ -7,4 +7,3 @@ git fetch upstream "pull/$request_id/head"
 git checkout upstream/master -b "review-${request_id}"
 git reset --hard FETCH_HEAD
 git pull --rebase
-

tools/lib/css_parser.py

@@ -503,4 +503,3 @@ def tokenize(text):
         advance(len(s))
 
     return tokens
-

tools/lib/html_grep.py

@@ -61,4 +61,3 @@ def grep(fns, words):
     # type: (List[str], Set[str]) -> None
     grepper = Grepper(fns)
     grepper.grep(words)
-

tools/setup/postgres-init-dev-db

@@ -85,4 +85,3 @@ EOF
 sh "$(dirname "$0")/../../scripts/setup/flush-memcached"
 
 echo "Database created"
-

tools/show-profile-results.py

@@ -22,4 +22,3 @@ except:
 p = pstats.Stats(fn)
 p.strip_dirs().sort_stats('cumulative').print_stats(25)
 p.strip_dirs().sort_stats('time').print_stats(25)
-

tools/tests/test_css_parser.py

@@ -188,4 +188,3 @@ class ParserTestSadPath(unittest.TestCase):
             }'''
         error = 'Comments in selector section are not allowed'
         self._assert_error(my_css, error)
-

tools/webpack

@@ -36,4 +36,3 @@ if args.watch:
     run_watch(args.port)
 else:
     run()
-

zerver/lib/export.py

@@ -1587,4 +1587,3 @@ def import_attachments(data):
         cursor.executemany(sql_template, tups)
 
     logging.info('Successfully imported M2M table %s' % (m2m_table_name,))
-

zerver/lib/html_diff.py

@@ -127,4 +127,3 @@ def highlight_html_differences(s1, s2):
         return s2
 
     return retval
-

zerver/lib/mandrill_client.py

@@ -15,4 +15,3 @@ def get_mandrill_client():
         MAIL_CLIENT = mandrill.Mandrill(settings.MANDRILL_API_KEY)
 
     return MAIL_CLIENT
-

zerver/lib/migrate.py

@@ -122,4 +122,3 @@ def act_on_message_ranges(db, orm, tasks, batch_size=5000, sleep=0.5):
 
         min_id = upper + 1
         time.sleep(sleep)
-

zerver/lib/parallel.py

@@ -73,4 +73,3 @@ if __name__ == "__main__":
         print(jobs)
         print(expected_output)
         print(output)
-

zerver/management/commands/fill_memcached_caches.py

@@ -19,4 +19,3 @@ class Command(BaseCommand):
 
         for cache in cache_fillers.keys():
             fill_remote_cache(cache)
-

zerver/management/commands/generate_invite_links.py

@@ -63,4 +63,3 @@ class Command(BaseCommand):
                 prereg_user = PreregistrationUser(email=email)
             prereg_user.save()
             print(email + ": " + Confirmation.objects.get_link_for_object(prereg_user))
-

zerver/tests/markdown/test_markdown.md

@@ -2,5 +2,3 @@
 
 
 This is some *bold text*.
-
-

zerver/tests/test_export.py

@@ -310,4 +310,3 @@ class ExportTest(TestCase):
         dummy_user_emails = get_set('zerver_userprofile_mirrordummy', 'email')
         self.assertIn('iago@zulip.com', dummy_user_emails)
         self.assertNotIn('cordelia@zulip.com', dummy_user_emails)
-

zerver/tests/test_external.py

@@ -126,4 +126,3 @@ class RateLimitTests(ZulipTestCase):
         result = self.send_api_message(email, api_key, "Good message")
 
         self.assert_json_success(result)
-

zerver/tests/test_presence.py

@@ -182,4 +182,3 @@ class UserPresenceTests(ZulipTestCase):
         # We only want @zulip.com emails
         for email in json['presences'].keys():
             self.assertEqual(split_email_to_domain(email), 'zulip.com')
-

zerver/tests/test_tutorial.py

@@ -80,4 +80,3 @@ class TutorialTests(ZulipTestCase):
             self.assert_json_success(result)
             user = get_user_profile_by_email(email)
             self.assertEqual(user.tutorial_status, expected_db_status)
-

zerver/tests/test_unread.py

@@ -250,4 +250,3 @@ class UnreadCountTests(ZulipTestCase):
                                                            "topic_name": invalid_topic_name,
                                                            "stream_name": "Denmark"})
         self.assert_json_error(result, 'No such topic \'abc\'')
-

zerver/views/pointer.py

@@ -52,4 +52,3 @@ def get_profile_backend(request, user_profile):
         result['max_message_id'] = messages[0].id
 
     return json_success(result)
-

zerver/views/tutorial.py

@@ -41,4 +41,3 @@ def json_tutorial_status(request, user_profile,
     user_profile.save(update_fields=["tutorial_status"])
 
     return json_success()
-

zilencer/tests.py

@@ -19,4 +19,3 @@ class EndpointDiscoveryTest(TestCase):
         data = ujson.loads(response.content)
         self.assertEqual(data["result"]["base_site_url"], "https://zulip.com/")
         self.assertEqual(data["result"]["base_api_url"], "https://api.zulip.com/")
-