Commit Graph

4979 Commits

Author SHA1 Message Date
Pavel Djundik
a7bd40a5b1 Fix users not being linked in /me actions 2017-11-30 14:13:11 +02:00
Pavel Djundik
104e270882
Merge pull request #1771 from thelounge/astorije/fix-user-in-msg
Clone instances of `User` in `Msg` to avoid unintentional mutations
2017-11-30 11:29:23 +02:00
Pavel Djundik
f89054784f
Merge pull request #1764 from thelounge/xpaw/fix-data-from
Fix data-from on messages
2017-11-30 11:28:00 +02:00
Pavel Djundik
be6975cb83
Merge pull request #1772 from thelounge/astorije/fix-nick-change
Fix nick changes wrongly reported
2017-11-30 11:26:26 +02:00
Pavel Djundik
bdbdb69f02
Merge pull request #1773 from thelounge/astorije/fix-default-from
Fix default value given to `Msg.from`
2017-11-30 11:25:30 +02:00
Jérémie Astori
fd72179b3d
Merge pull request #1777 from thelounge/greenkeeper/eslint-4.12.1
Update eslint to the latest version 🚀
2017-11-30 02:12:15 -05:00
greenkeeper[bot]
2f4205d095 chore(package): update eslint to version 4.12.1 2017-11-30 06:04:42 +00:00
Jérémie Astori
4ec10b922a
Clone instances of User in Msg to avoid unintentional mutations 2017-11-30 00:41:19 -05:00
Jérémie Astori
e60bbd9548
Fix nick changes wrongly reported 2017-11-29 18:44:33 -05:00
Jérémie Astori
c0d348a678
Fix default value given to Msg.from
- `{}` is not falsey so Handlebars would try to render the block (see http://handlebarsjs.com/builtin_helpers.html#conditionals), therefore with a `nick` of `undefined`, which breaks `colorClass` (doing `undefined.length)
- There does not seem to be a way to check for empty objects in Handlebars (sigh) so checking `from.nick` seems like the most reliable way to check for a non-empty value. Alternatively, we could use a helper to check `{}` but meh.
2017-11-29 18:37:17 -05:00
Pavel Djundik
36e0ce46b4
Merge pull request #1775 from thelounge/greenkeeper/moment-2.19.3
Update moment to the latest version 🚀
2017-11-29 22:25:02 +02:00
Pavel Djundik
68753ae3e8 Fix data-from on messages 2017-11-29 20:45:19 +02:00
Pavel Djundik
53968bf453
Merge pull request #1769 from thelounge/astorije/test-timeout
Increase timeout of server tests
2017-11-29 18:45:57 +02:00
greenkeeper[bot]
a97a6de60f fix(package): update moment to version 2.19.3 2017-11-29 16:37:20 +00:00
Jérémie Astori
7de7292560
Increase timeout of server tests 2017-11-28 19:43:00 -05:00
Pavel Djundik
c9e340e153
Merge pull request #1761 from thelounge/astorije/slow-tests
Mark slow tests as such to reduce noise on test report
2017-11-28 11:43:24 +02:00
Pavel Djundik
c3b076ca74
Merge pull request #1759 from thelounge/astorije/avoid-escape-quotes2
Avoid escaping quotes whenever possible (again)
2017-11-28 11:41:15 +02:00
Pavel Djundik
561de89568
Merge pull request #1763 from thelounge/greenkeeper/commander-2.12.2
Update commander to the latest version 🚀
2017-11-28 11:41:02 +02:00
greenkeeper[bot]
12e0087d47 fix(package): update commander to version 2.12.2 2017-11-28 07:14:48 +00:00
Jérémie Astori
624b3ebc18
Mark slow tests as such to reduce noise on test report 2017-11-27 18:47:19 -05:00
Jérémie Astori
03d6bf06ee
Avoid escaping quotes whenever possible (again)
Oops, I forgot to rebase and fix after merging a previous PR. I wish there was a way to enforce this with ESLint, but did not find any (only allow) :(
2017-11-27 18:40:23 -05:00
Jérémie Astori
c9da29f2fc
Merge pull request #1757 from thelounge/xpaw/detached-sign-in
Do not keep sign-in and loader references in memory
2017-11-27 18:32:20 -05:00
Jérémie Astori
7daf566025
Merge pull request #1758 from MaxLeiter/improve-regex
update nick regex for linking nicks in text
2017-11-27 18:30:37 -05:00
Max Leiter
f98deaba70 update nick regex for linking nicks in text 2017-11-27 13:58:10 -08:00
Jérémie Astori
74521225ba
Merge pull request #1751 from thelounge/greenkeeper/stylelint-8.3.1
Update stylelint to the latest version 🚀
2017-11-27 13:32:01 -05:00
Jérémie Astori
b53f1719eb
Merge pull request #1749 from thelounge/astorije/avoid-escape-quotes
Avoid escaping quotes whenever possible
2017-11-27 12:57:55 -05:00
Jérémie Astori
e5862a9303
Merge pull request #1752 from thelounge/xpaw/simple-build-cmd
Avoid using npm-run-all for build
2017-11-27 12:57:21 -05:00
Jérémie Astori
39e99fcd29
Merge pull request #1753 from thelounge/xpaw/webp
Detect image/webp as an image
2017-11-27 12:55:27 -05:00
Pavel Djundik
5855099d5b Do not keep sign-in and loader references in memory 2017-11-27 19:39:16 +02:00
Pavel Djundik
4e2eed2023
Merge pull request #1709 from MaxLeiter/master
Link nicks mentioned in messages
2017-11-27 19:31:36 +02:00
Max Leiter
3d31fa4686 Link nicks mentioned in messages 2017-11-27 09:10:56 -08:00
Pavel Djundik
4e45820235
Detect image/webp as an image 2017-11-27 15:50:35 +02:00
Pavel Djundik
ba002cca64
Merge pull request #1708 from thelounge/astorije/thelounge
Switch CLI tool from `lounge` to `thelounge`, deprecate `lounge`
2017-11-27 13:58:46 +02:00
Pavel Djundik
6fc812d272
Merge pull request #1748 from MaxLeiter/highlights
Fix text highlighting when clicking on condensed message toggles
2017-11-27 11:48:03 +02:00
Pavel Djundik
0dbdd4732b
Avoid using npm-run-all for build 2017-11-27 11:45:58 +02:00
Pavel Djundik
52f8934d9b
Merge pull request #1750 from thelounge/astorije/parallelize-npm-test
Parallelize `npm test`
2017-11-27 11:44:51 +02:00
greenkeeper[bot]
2fcb89a779 chore(package): update stylelint to version 8.3.1 2017-11-27 08:59:26 +00:00
Jérémie Astori
b25884c5bd
Make stylelint report unncessary disables
From the man:

```
    --report-needless-disables, --rd

      Report stylelint-disable comments that are not blocking a lint warning.
      If you provide the argument "error", the process will exit with code 2
      if needless disables are found.
```
2017-11-26 17:57:14 -05:00
Jérémie Astori
2a8feaa454
Parallelize npm test
After this, tests run in ~7s locally vs. ~14s without. ¯\_(ツ)_/¯
One caveat is that coloring must be forced now. Hope this looks okay on Travis.
2017-11-26 17:54:58 -05:00
Jérémie Astori
287c3a8223
Avoid escaping quotes whenever possible
Our `.eslintrc.yml` configuration file already allows for avoiding escape (see [ESLint doc for `avoidEscape`](https://eslint.org/docs/rules/quotes#avoidescape)) so we might as well use it. We already use this in a few places I believe.
2017-11-26 17:34:28 -05:00
Jérémie Astori
823ed0153f
Merge pull request #1747 from thelounge/astorije/fix-parse
Fix duplicate text generated when sending channel and link together
2017-11-26 17:23:32 -05:00
Max Leiter
d9d4672d75 Fix text highlighting when clicking on condensed message toggles 2017-11-26 14:05:23 -08:00
Jérémie Astori
ddc7ace78d
Bring test from ircmessageparser
See https://github.com/Bonuspunkt/ircmessageparser/pull/12/files#diff-e6c9a6bca996bc454cc244d17bfeda5c for reference (same test, linted)
2017-11-26 17:02:57 -05:00
Bonuspunkt
0e52b133af Sending channel and link together generates duplicate text 2017-11-26 21:33:27 +02:00
Jérémie Astori
d9efaef369
Merge pull request #1746 from thelounge/xpaw/fix-preview-keep-to-bottom
Trigger keep to bottom for previews correctly
2017-11-26 12:35:08 -05:00
Jérémie Astori
7f16ad649d
Merge pull request #1745 from thelounge/greenkeeper/stylelint-8.3.0
Update stylelint to the latest version 🚀
2017-11-26 12:33:08 -05:00
Pavel Djundik
6862c2e6b0 Trigger keep to bottom for previews correctly 2017-11-26 15:49:31 +02:00
Pavel Djundik
e3515d2ea1 Update stylelint rules 2017-11-26 14:28:33 +02:00
greenkeeper[bot]
645b3c13b3 chore(package): update stylelint-config-standard to version 18.0.0 2017-11-26 14:06:27 +02:00
greenkeeper[bot]
c5a530fe31 chore(package): update stylelint to version 8.3.0 2017-11-26 10:54:28 +00:00