This compiles without error, and at initial testing seems to be working alright, but of course I'd need to keep a close eye on it.


What good is this when there isn't a MCMMO that works with 1.9 on every single block place/change event ? Isn't there a mcmmo.jar that works with 1.9 ?


Okay, a few remarks at the overall level.

* The acceptance tests need to start from the "top", so the first one should be `Get tab stops object` (from a paragraph format object).

  * This should be pretty much exactly analogous to `Get paragraph format object` in par-access-parfmt.feature.
  * It should be added to txt-parfmt-props.feature
  * It should have a case for a paragraph format having tab stops (say three) and one *not* having tabs stops. In either case it should return a TabStops object.

* At this point, we have a valid TabStops object and can proceed to the next level, that TabStops has the sequence behaviors we expect, specifically len(), iteratation, and indexed access.

  * I'm thinking we start a new feature file for tabs, maybe two. They get a little cumbersome once they get more than a half-dozen scenarios or so and it looks like there will be plenty here. So instead of 'par-tab-stops.feature' let's start with 'tab-access-tabs.feature'.
  * This file should be pretty much analogous to sty-access-latent-styles.feature, and exercise len(), iteration, and indexed access.
  * The scenarios for add and delete should go here too, since those are operations on the collection, not an individual tab.

* Then I'm thinking a separate file for TabStop level operations, we can call that one 'tab-tab-props.feature' all the scenarios that get or set Tab properties should go there. So any that start with "Given a tab stop ...".

I'll take a look at the details and see if I have any inline remarks ...


#5536-AirConditionerVariableRefrigerantFlow-with-ThermostatOffsetPriority-crashes-with-blank-Zone-Name-for-Master-Thermostat-Location (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Unless we're aiming for IE 8 and below, we can just use `.trim()`. And `searchBox` seems to only be used here, how about `$("#searchToolSelectionBox").val().trim().toLowerCase()` ?


Validated the changes, have been using it since in `vkernel` and `rump2cos` projects.


To build AWS infrastructure for a 3 tier Web Application Architecture. The tiers that comprise this architecture include the Web tier, the Application tier, and the Database tier. The Web tier represents the component that users directly interact with (such as a web page, mobile app UI, etc.). The Application tier contains the code required to translate user actions at the Web tier to the functionality that drives the application’s behavior. The data tier consists of storage media (databases, object stores, caches, file systems, etc.) that hold the data relevant to the application.

This script supports following features.

VPC and subnet details such as CIDR, Availability zone, Resource name, etc. need to be an input to the script,which is entered in Dynamo DB tables. Scripts read Dynamo DB tables and create the AWS resources such as VPC, Subnets, NACLs and Security Groups.
Subnet’s in each layer is protected by appropriate NACLS. Configure security groups to protect resources in each tier.
Web tier and App tier have Auto scaling groups and it spreads across 2 AZ's. External load balancer controls traffic to Web Tier and the Internal Load balancer controls the traffic to the App tier.

The instances in the App Tier can access the Internet by using a network address translation (NAT) gateway that resides in the Management subnet. The database servers can connect to the Internet for software updates using the NAT gateway, but the Internet cannot initiate connections to the database servers.


change to `POST`


Ah, yeah,  I was thinking of `$(CODE_DIR)`...whoops


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18963](https://teamcity.mesosphere.io/viewLog.html?buildId=162107&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:49


I've moved recently to making these 'Then' statements more like the code, since the API is the user interface for a library. So this would become: "Then tab_stop.alignment is <value>". Note this leverages the naming convention that an object instance variable is snake-case version of the object name (TabStop -> tab_stop in this case).


Ich hab dass wort vergessen


@jcalovski Just for curiosity, why did you run (and why where you allowed to do so) this with [0,4] bounds?


This patch killed the baro sim. @julianoes Did you request this change based on in-flight observations?


ea5d3ecc25d511da47a9b2dd6d27e9ab756974b4 fixed. Thank you!


It looks like Qt 5.7 will add `qAsConst()` just for that (see [New Features in Qt 5.7](https://wiki.qt.io/New_Features_in_Qt_5.7)) so I think it's better to define it ourselves on pre-5.7 and use it (or just add `const` if possible) where appropriate.


all doc comments need to be aligned


same


this is one empty line too much


After your point on `AgIgnore` I like idea to append to list, but don't like overbearing `ag#opts#append` nevertheless. Because lists are needed only for several commands at most. Therefore it's better to do this in similar way to `AgIgnore` -- direct option for appending to each of that commands instead of generalized `AgAppend`. Currently there are only two other such api: `AgAdd` and `AgPaths`.


the shortlog could be made a bit more informative, how about something like `setup: Move up version assert` or something like that? That's what you're doing, not "changing imports" randomly right :)


You mean in every method?


Line is longer than allowed. (83 > 80)

*LineLengthBear, severity NORMAL, section `linelength`.*


Line is longer than allowed. (81 > 80)

*LineLengthBear, severity NORMAL, section `linelength`.*


you seem to not set `end_line` although the info is given in the JSON!


Oh well. It doesn't hurt to tell.


Alright, let `AgReset` to be. But instead of empty value, which you can set through `AgSet`, it's more reasonable to reset to corresponding `s:ag.*` value. Therefore if you have original `g:ag.ignore = ['tags']` you could after appending reset back to default value.


I'd use `origin="{} ({})".format(self.__class__.__name__, issue['ruleName']`, that'll give the user more precise info


I still want to delete AgShow


Thanks for the fixes, but this should definitely be a long, not an int.

> should just be an "int" I guess, so it uses the correct 32/64 bit int type accordingly to pointer size

ints are always 32-bit in C#, regardless of pointer width (they are the same as System.Int32.) The only variable width integer types in C# are IntPtr and UIntPtr, though that doesn't really matter in this case, because mpv is always expecting a 64-bit int, even on 32-bit architectures.

The following should give the right type:

```c#
var windowId = pictureBox1.Handle.ToInt64();
```

The definition of _mpvSetOption should also be changed to use a ``ref long`` parameter. Longs are always 64-bit in C# (they are the same as System.Int64.)


There is an enormous amount of whitespace diff-noise in web.config which is caused by something rewriting it - do we want that on this commit?


this commit is for #47


Btw, qPow doesn't have this problem and works without any casting though consistency would be a good thing (TM) and not all cmath functions are provided by QtMath.


Line contains following spacing inconsistencies:
- Trailing whitespaces.

*SpaceConsistencyBear, severity NORMAL, section `python`.*


@delta24
I think that I may have a solution. The curl command has a flag ( -w) that allows to print some valuable information , like the http-code-response , the total time, the upload time. And in this case that we are using random information to put (from /dev/random), the -w flag with some variables will give us more general information.

i.e.

#FILE=$(head /dev/random -c8 | od -tx1 -w16 | head -n1 | cut -d' ' -f2- | tr -d ' '); curl -s -i -A 'Mozilla/5.0 (X11; Linux i686; rv:6.0) Gecko/20100101 Firefox/15.0' -X 'PUT' -w \n\n\nhttp-response-code:%{http_code}\nRemote-IP:%{remote_ip}\nRemote-Port%{remote_port}\nUploaded-size:%{size_upload}\nSize-Downloaded:%{size_download}\nConnect-Time:%{time_connect}s\nTotal-time:%{time_total}s\nEffective-url:%{url_effective}\n --data "$FILE" -k "https://target.com/admin/$FILE" 2> /dev/null

The part of the output will be:
http-response-code:501
Remote-IP:104.85.166.89
Remote-Port443
Uploaded-size:16
Size-Downloaded:257
Connect-Time:0.074s
Total-time:0.612s
Effective-url:https://target.com/admin/66c00f5789927587


You should add comments where you change things in the Maven code, e.g. `// plugin-meta start`, or `// plugin-meta` (at the end of the changed modifier lines), so we can easily update it for a new version from Maven later.


you ignored my comment here again... is it that difficult? It's very simple: look at *every single comment* and either do a correction on that place *or explain* why the comment isn't valid. This is very frustrating for the reviewer! I'm nowadays spending my whole day reviewing and can't code because of that.


Should be run a `println!` from inside the library? Or just return the strings?


If we need to do ToList() (not sure we do, because it's only a hashset), we should do it before the .Count() - if it's expensive to walk that collection, we don't want to do it twice.

I am not really sure what's going on here with the anon-compare - why do we need to do a reflection-based compare across this stuff?


[inst/tests/test-brew.R:6:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L6) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(capture.output(Pandoc.brew(text='## foo')), '## foo ')
                                               ~^~
```
[inst/tests/test-brew.R:7:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L7) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(capture.output(Pandoc.brew(text='## <%=1%>')), '## _1_ ')
                                               ~^~
```
[inst/tests/test-brew.R:7:52:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L7) *style:* **Commented code should be removed.**
```r
    expect_equal(capture.output(Pandoc.brew(text='## <%=1%>')), '## _1_ ')
                                                   ^~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:8:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L8) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(capture.output(Pandoc.brew(text='<%=1%>')), '_1_ ')
                                               ~^~
```
[inst/tests/test-brew.R:9:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L9) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(capture.output(Pandoc.brew(text='<%for (i in 1:3)%><%=i%>')), '_1__2__3_ ')
                                               ~^~
```
[inst/tests/test-brew.R:15:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L15) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(length(Pandoc.brew(text='## foo', output = output)), 1)
                                       ~^~
```
[inst/tests/test-brew.R:15:44:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L15) *style:* **Commented code should be removed.**
```r
    expect_equal(length(Pandoc.brew(text='## foo', output = output)), 1)
                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:16:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L16) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(length(Pandoc.brew(text='## foo <%=mtcars[1:2,]%>', output = output)), 3)
                                       ~^~
```
[inst/tests/test-brew.R:16:44:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L16) *style:* **Commented code should be removed.**
```r
    expect_equal(length(Pandoc.brew(text='## foo <%=mtcars[1:2,]%>', output = output)), 3)
                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:17:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L17) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(length(Pandoc.brew(text='## foo <%=mtcars[1:2,]%> bar <%=plot(1:10)%>', output = output)), 5)
                                       ~^~
```
[inst/tests/test-brew.R:17:44:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L17) *style:* **Commented code should be removed.**
```r
    expect_equal(length(Pandoc.brew(text='## foo <%=mtcars[1:2,]%> bar <%=plot(1:10)%>', output = output)), 5)
                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:18:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L18) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(length(Pandoc.brew(text='## foo <%=1%> bar <%=pi%>', output = output)), 1)
                                       ~^~
```
[inst/tests/test-brew.R:18:44:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L18) *style:* **Commented code should be removed.**
```r
    expect_equal(length(Pandoc.brew(text='## foo <%=1%> bar <%=pi%>', output = output)), 1)
                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:19:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L19) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(length(Pandoc.brew(text='<%for (i in 1:5){%>## foo<%}%>', output = output)), 5)
                                       ~^~
```
[inst/tests/test-brew.R:19:63:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L19) *style:* **Commented code should be removed.**
```r
    expect_equal(length(Pandoc.brew(text='<%for (i in 1:5){%>## foo<%}%>', output = output)), 5)
                                                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:20:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L20) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(length(Pandoc.brew(text='# FOO <%=pi%> bar \n# sad', output = output)), 2)
                                       ~^~
```
[inst/tests/test-brew.R:21:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L21) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(length(Pandoc.brew(text='# FOO\n## bar<%=pi%> bar \n# sad', output = output)), 3)
                                       ~^~
```
[inst/tests/test-brew.R:22:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L22) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(length(Pandoc.brew(text='# FOO ## bar<%=pi%> barn# sad', output = output)), 1)
                                       ~^~
```
[inst/tests/test-brew.R:22:45:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L22) *style:* **Commented code should be removed.**
```r
    expect_equal(length(Pandoc.brew(text='# FOO ## bar<%=pi%> barn# sad', output = output)), 1)
                                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:29:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L29) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='foo <%=1:10%>', output = output)), 'text')
                                              ~^~
```
[inst/tests/test-brew.R:30:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L30) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='## foo <%=1:10%>', output = output)), 'heading')
                                              ~^~
```
[inst/tests/test-brew.R:30:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L30) *style:* **Commented code should be removed.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='## foo <%=1:10%>', output = output)), 'heading')
                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:31:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L31) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='foo\n<%=1:10%>', output = output)), c('text', 'block', 'text'))
                                              ~^~
```
[inst/tests/test-brew.R:32:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L32) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='## foo\n<%=1:10%>', output = output)), c('heading', 'block', 'text'))
                                              ~^~
```
[inst/tests/test-brew.R:32:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L32) *style:* **Commented code should be removed.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='## foo\n<%=1:10%>', output = output)), c('heading', 'block', 'text'))
                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:33:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L33) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='foo <%=list(1:10)%>', output = output)), c('text', 'block', 'text'))
                                              ~^~
```
[inst/tests/test-brew.R:34:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L34) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='## foo <%=mtcars%>', output = output)), c('heading', 'block', 'text'))
                                              ~^~
```
[inst/tests/test-brew.R:34:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L34) *style:* **Commented code should be removed.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='## foo <%=mtcars%>', output = output)), c('heading', 'block', 'text'))
                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:35:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L35) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='foo has <%=1%> apple and <%=3%> pears', output = output)), 'text')
                                              ~^~
```
[inst/tests/test-brew.R:36:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L36) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='foo has \nsome neat apples. Exactly <%=5%>', output = output)),
                                              ~^~
```
[inst/tests/test-brew.R:38:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L38) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='<%=5%> apple', output = output)), 'text')
                                              ~^~
```
[inst/tests/test-brew.R:39:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L39) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='## header\n<%=5%> apple', output = output)), c('heading', 'text'))
                                              ~^~
```
[inst/tests/test-brew.R:39:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L39) *style:* **Commented code should be removed.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='## header\n<%=5%> apple', output = output)), c('heading', 'text'))
                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-brew.R:40:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-brew.R#L40) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(getChunkTypes(Pandoc.brew(text='<%=plot(1:10)%> ', output = output)), c('block', 'text'))
                                              ~^~
```
[inst/tests/test-convert.R:6:40:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-convert.R#L6) *style:* **Put spaces around all infix operators.**
```r
    tf <- paste(tempfile(), '.txt', sep='')
                                      ~^~
```
[inst/tests/test-convert.R:10:6:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-convert.R#L10) *style:* **Commented code should be removed.**
```r
#    expect_equal(openFileInOS(tf), 0)
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-evals.R:60:36:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-evals.R#L60) *style:* **Put spaces around all infix operators.**
```r
    expect_error(evals('1:5', hooks=1:10))
                                  ~^~
```
[inst/tests/test-evals.R:96:25:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-evals.R#L96) *style:* **Commented code should be removed.**
```r
    expect_null(evals('## comment')[[1]]$result)
                        ^~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-evals.R:100:32:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-evals.R#L100) *style:* **Commented code should be removed.**
```r
    expect_null(evals(list(c('## comment', 'y <- rnorm(10)')), parse = F)[[1]]$result)
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-evals.R:150:13:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-evals.R#L150) *style:* **Commented code should be removed.**
```r
test_that('## comment', {
            ^~~~~~~~~~~~~
```
[inst/tests/test-evals.R:151:34:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-evals.R#L151) *style:* **Commented code should be removed.**
```r
    expect_is(evals('rnorm(100);## comment')[[1]]$result, 'numeric')
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-evals.R:152:34:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-evals.R#L152) *style:* **Commented code should be removed.**
```r
    expect_is(evals('rnorm(100);## comment', parse = FALSE)[[1]]$result, 'numeric')
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-evals.R:276:17:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-evals.R#L276) *style:* **Put spaces around all infix operators.**
```r
    png(filename=file.path(dir, 'recorded.png'))
               ~^~
```
[inst/tests/test-helpers.R:24:78:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-helpers.R#L24) *style:* **Commas should always have a space after.**
```r
test_that('splitLine leaves non-string arguments and empty string unchanged',{
                                                                             ^
```
[inst/tests/test-helpers.R:35:59:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-helpers.R#L35) *style:* **Commas should always have a space after.**
```r
test_that('splitLine without hyphening behaves correctly',{
                                                          ^
```
[inst/tests/test-helpers.R:50:62:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-helpers.R#L50) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(splitLine(x, 0), paste(strwrap(x, 0), collapse='\n')) # zero width replaces whitespaces by line breaks
                                                            ~^~
```
[inst/tests/test-helpers.R:61:56:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-helpers.R#L61) *style:* **Commas should always have a space after.**
```r
test_that('splitLine with hyphening behaves correctly',{
                                                       ^
```
[inst/tests/test-missing.R:13:50:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-missing.R#L13) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(paste(pander_return(data.frame(a=1:2, b=c(1, NA), c=c(NA, 'z'))), collapse = '\n'),
                                                ~^~
```
[inst/tests/test-missing.R:13:57:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-missing.R#L13) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(paste(pander_return(data.frame(a=1:2, b=c(1, NA), c=c(NA, 'z'))), collapse = '\n'),
                                                       ~^~
```
[inst/tests/test-missing.R:13:69:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-missing.R#L13) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(paste(pander_return(data.frame(a=1:2, b=c(1, NA), c=c(NA, 'z'))), collapse = '\n'),
                                                                   ~^~
```
[inst/tests/test-missing.R:22:50:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-missing.R#L22) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(paste(pander_return(data.frame(a=1:2, b=c(1, NA), c=c(NA, 'z'))), collapse = '\n'),
                                                ~^~
```
[inst/tests/test-missing.R:22:57:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-missing.R#L22) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(paste(pander_return(data.frame(a=1:2, b=c(1, NA), c=c(NA, 'z'))), collapse = '\n'),
                                                       ~^~
```
[inst/tests/test-missing.R:22:69:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-missing.R#L22) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(paste(pander_return(data.frame(a=1:2, b=c(1, NA), c=c(NA, 'z'))), collapse = '\n'),
                                                                   ~^~
```
[inst/tests/test-S3.R:58:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L58) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(nchar('Δ Δ Δ Δ Δ', type='width'), 9)
                                       ~^~
```
[inst/tests/test-S3.R:59:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L59) *style:* **Put spaces around all infix operators.**
```r
    res_grid <- pander_return(t, style='grid', split.tables = 23)
                                     ~^~
```
[inst/tests/test-S3.R:62:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L62) *style:* **Put spaces around all infix operators.**
```r
    res_grid <- pander_return(t, style='grid', split.tables = 22)
                                     ~^~
```
[inst/tests/test-S3.R:65:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L65) *style:* **Put spaces around all infix operators.**
```r
    res_simple <- pander_return(t, style='simple', split.tables = 17)
                                       ~^~
```
[inst/tests/test-S3.R:68:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L68) *style:* **Put spaces around all infix operators.**
```r
    res_simple <- pander_return(t, style='simple', split.tables = 16)
                                       ~^~
```
[inst/tests/test-S3.R:75:34:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L75) *style:* **Put spaces around all infix operators.**
```r
    res <- pander_return(d, style='rmarkdown')
                                ~^~
```
[inst/tests/test-S3.R:80:34:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L80) *style:* **Put spaces around all infix operators.**
```r
    res <- pander_return(d, style='rmarkdown')
                                ~^~
```
[inst/tests/test-S3.R:89:42:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L89) *style:* **Commas should always have a space after.**
```r
    expect_warning(pander_return(mtcars[,1:2], split.tables = Inf, split.cells = c('50%', '50%')))
                                         ^
```
[inst/tests/test-S3.R:100:42:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L100) *style:* **Put spaces around all infix operators.**
```r
    expect_error(pander_return(t, justify='laft'))
                                        ~^~
```
[inst/tests/test-S3.R:155:58:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L155) *style:* **Commas should always have a space after.**
```r
    expect_that(pandoc.table(data.frame(x = 1:2, y = c(1,NA)), emphasize.italics.cols = 2), not(gives_warning()))
                                                         ^
```
[inst/tests/test-S3.R:160:23:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L160) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(a=1:3, b=1:3, c=1:3)
                     ~^~
```
[inst/tests/test-S3.R:160:30:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L160) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(a=1:3, b=1:3, c=1:3)
                            ~^~
```
[inst/tests/test-S3.R:160:37:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L160) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(a=1:3, b=1:3, c=1:3)
                                   ~^~
```
[inst/tests/test-S3.R:161:67:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L161) *style:* **Commas should always have a space after.**
```r
    res <- capture.output(pander(df, emphasize.italics.rows = c(1,2), style = 'simple'))
                                                                  ^
```
[inst/tests/test-S3.R:164:66:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L164) *style:* **Commas should always have a space after.**
```r
    res <- capture.output(pander(df, emphasize.strong.rows = c(1,2), style = 'simple'))
                                                                 ^
```
[inst/tests/test-S3.R:170:23:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L170) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(a=1:3, b=4:6, c=7:9)
                     ~^~
```
[inst/tests/test-S3.R:170:30:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L170) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(a=1:3, b=4:6, c=7:9)
                            ~^~
```
[inst/tests/test-S3.R:170:37:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L170) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(a=1:3, b=4:6, c=7:9)
                                   ~^~
```
[inst/tests/test-S3.R:171:60:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L171) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(df, emphasize.verbatim.rows = c(1,2), style = 'simple')
                                                           ^
```
[inst/tests/test-S3.R:174:60:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L174) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(df, emphasize.verbatim.cols = c(1,2), style = 'simple')
                                                           ^
```
[inst/tests/test-S3.R:181:54:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L181) *style:* **Commas should always have a space after.**
```r
                         emphasize.strong.rows = c(1,2), style = 'simple')
                                                     ^
```
[inst/tests/test-S3.R:268:42:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L268) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(m, digits = c(2,4,1))
                                         ^
```
[inst/tests/test-S3.R:268:44:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L268) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(m, digits = c(2,4,1))
                                           ^
```
[inst/tests/test-S3.R:270:43:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L270) *style:* **Put spaces around all infix operators.**
```r
    expect_warning(pander_return(m, digits=c(1,2)))
                                         ~^~
```
[inst/tests/test-S3.R:270:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L270) *style:* **Commas should always have a space after.**
```r
    expect_warning(pander_return(m, digits=c(1,2)))
                                               ^
```
[inst/tests/test-S3.R:272:25:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L272) *style:* **Commas should always have a space after.**
```r
    expect_equal(res[3],'0.11  0.11  0.11 ')
                        ^
```
[inst/tests/test-S3.R:274:43:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L274) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(mt, digits = c(1,4,3,4), keep.trailing.zeros = TRUE)
                                          ^
```
[inst/tests/test-S3.R:274:45:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L274) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(mt, digits = c(1,4,3,4), keep.trailing.zeros = TRUE)
                                            ^
```
[inst/tests/test-S3.R:274:47:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L274) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(mt, digits = c(1,4,3,4), keep.trailing.zeros = TRUE)
                                              ^
```
[inst/tests/test-S3.R:280:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L280) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(m, round = c(2,4,1))
                                        ^
```
[inst/tests/test-S3.R:280:43:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L280) *style:* **Commas should always have a space after.**
```r
    res <- pander_return(m, round = c(2,4,1))
                                          ^
```
[inst/tests/test-S3.R:282:42:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L282) *style:* **Put spaces around all infix operators.**
```r
    expect_warning(pander_return(m, round=c(1,2)))
                                        ~^~
```
[inst/tests/test-S3.R:282:47:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L282) *style:* **Commas should always have a space after.**
```r
    expect_warning(pander_return(m, round=c(1,2)))
                                              ^
```
[inst/tests/test-S3.R:284:25:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L284) *style:* **Commas should always have a space after.**
```r
    expect_equal(res[3],'0.11  0.11  0.11 ')
                        ^
```
[inst/tests/test-S3.R:290:20:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L290) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a='Pander\nPackage')
                  ~^~
```
[inst/tests/test-S3.R:296:20:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L296) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a=c('Pander\nPackage','Pander\nPackage'))
                  ~^~
```
[inst/tests/test-S3.R:296:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L296) *style:* **Commas should always have a space after.**
```r
  x <- data.frame(a=c('Pander\nPackage','Pander\nPackage'))
                                        ^
```
[inst/tests/test-S3.R:302:20:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L302) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a=c('Pander Package','Pander Package'))
                  ~^~
```
[inst/tests/test-S3.R:302:40:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L302) *style:* **Commas should always have a space after.**
```r
  x <- data.frame(a=c('Pander Package','Pander Package'))
                                       ^
```
[inst/tests/test-S3.R:309:20:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L309) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a=rep('Pander\nPackage', rows))
                  ~^~
```
[inst/tests/test-S3.R:316:20:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L316) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a=paste(rep('pander', n), collapse='\n'))
                  ~^~
```
[inst/tests/test-S3.R:316:53:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L316) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a=paste(rep('pander', n), collapse='\n'))
                                                   ~^~
```
[inst/tests/test-S3.R:323:20:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L323) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a=paste(rep('pander', n[1]), collapse='\n'),
                  ~^~
```
[inst/tests/test-S3.R:323:56:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L323) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a=paste(rep('pander', n[1]), collapse='\n'),
                                                      ~^~
```
[inst/tests/test-S3.R:324:58:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L324) *style:* **Put spaces around all infix operators.**
```r
                  b = paste(rep('pander', n[2]), collapse='\n'),
                                                        ~^~
```
[inst/tests/test-S3.R:325:58:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L325) *style:* **Put spaces around all infix operators.**
```r
                  c = paste(rep('pander', n[3]), collapse='\n'))
                                                        ~^~
```
[inst/tests/test-S3.R:331:16:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L331) *style:* **Commas should always have a space after.**
```r
  x <- rbind(x,x)
               ^
```
[inst/tests/test-S3.R:338:41:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L338) *style:* **Commas should always have a space after.**
```r
test_that('split.cells works correctly',{
                                        ^
```
[inst/tests/test-S3.R:355:57:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L355) *style:* **Commas should never have a space before.**
```r
  expect_equal(pandoc.table.return(x, split.cells = c(6 ,6)), pandoc.table.return(x, split.cells = 6))
                                                       ~^
```
[inst/tests/test-S3.R:355:58:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L355) *style:* **Commas should always have a space after.**
```r
  expect_equal(pandoc.table.return(x, split.cells = c(6 ,6)), pandoc.table.return(x, split.cells = 6))
                                                         ^
```
[inst/tests/test-S3.R:356:57:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L356) *style:* **Commas should never have a space before.**
```r
  expect_equal(pandoc.table.return(x, split.cells = c(7 ,7)), pandoc.table.return(x, split.cells = 7))
                                                       ~^
```
[inst/tests/test-S3.R:356:58:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L356) *style:* **Commas should always have a space after.**
```r
  expect_equal(pandoc.table.return(x, split.cells = c(7 ,7)), pandoc.table.return(x, split.cells = 7))
                                                         ^
```
[inst/tests/test-S3.R:357:57:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L357) *style:* **Commas should never have a space before.**
```r
  expect_equal(pandoc.table.return(x, split.cells = c(7 ,7)), pandoc.table.return(x, split.cells = 7))
                                                       ~^
```
[inst/tests/test-S3.R:357:58:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L357) *style:* **Commas should always have a space after.**
```r
  expect_equal(pandoc.table.return(x, split.cells = c(7 ,7)), pandoc.table.return(x, split.cells = 7))
                                                         ^
```
[inst/tests/test-S3.R:367:58:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L367) *style:* **Commas should always have a space after.**
```r
test_that('split.cells param produces expected warnings',{
                                                         ^
```
[inst/tests/test-S3.R:369:47:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L369) *style:* **Commas should always have a space after.**
```r
  expect_warning(pander(mt, split.cells = c(1,2)))
                                              ^
```
[inst/tests/test-S3.R:370:47:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L370) *style:* **Commas should always have a space after.**
```r
  expect_warning(pander(mt, split.cells = c(1,2,3)))
                                              ^
```
[inst/tests/test-S3.R:370:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L370) *style:* **Commas should always have a space after.**
```r
  expect_warning(pander(mt, split.cells = c(1,2,3)))
                                                ^
```
[inst/tests/test-S3.R:371:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L371) *style:* **Commas should always have a space after.**
```r
  expect_warning(pander(mt, split.cells = c('10%','10%','10%')))
                                                  ^
```
[inst/tests/test-S3.R:371:57:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L371) *style:* **Commas should always have a space after.**
```r
  expect_warning(pander(mt, split.cells = c('10%','10%','10%')))
                                                        ^
```
[inst/tests/test-S3.R:372:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L372) *style:* **Commas should always have a space after.**
```r
  expect_warning(pander(mt, split.cells = c('30%','30%','40%')))
                                                  ^
```
[inst/tests/test-S3.R:372:57:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L372) *style:* **Commas should always have a space after.**
```r
  expect_warning(pander(mt, split.cells = c('30%','30%','40%')))
                                                        ^
```
[inst/tests/test-S3.R:377:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L377) *style:* **Commas should always have a space after.**
```r
test_that('produces.errors',{
                            ^
```
[inst/tests/test-S3.R:378:20:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L378) *style:* **Put spaces around all infix operators.**
```r
  x <- data.frame(a='Pander\nPackage')
                  ~^~
```
[inst/tests/test-S3.R:379:31:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L379) *style:* **Put spaces around all infix operators.**
```r
  expect_error(pander(x, style='simple', keep.line.breaks = TRUE))
                             ~^~
```
[inst/tests/test-S3.R:380:31:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L380) *style:* **Put spaces around all infix operators.**
```r
  expect_error(pander(x, style='rmarkdown', keep.line.breaks = TRUE))
                             ~^~
```
[inst/tests/test-S3.R:387:44:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L387) *style:* **Commas should always have a space after.**
```r
test_that('table.expand behaves correctly',{
                                           ^
```
[inst/tests/test-S3.R:392:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L392) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:392:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L392) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:392:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L392) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:394:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L394) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)),
                                                      ^
```
[inst/tests/test-S3.R:397:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L397) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                  ^
```
[inst/tests/test-S3.R:397:114:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L397) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                                                ^
```
[inst/tests/test-S3.R:403:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L403) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:403:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L403) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:403:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L403) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:405:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L405) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)), length(strsplit(res, '\n')[[1]]))
                                                      ^
```
[inst/tests/test-S3.R:413:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L413) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:413:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L413) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:413:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L413) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:415:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L415) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)), length(strsplit(res, '\n')[[1]]))
                                                      ^
```
[inst/tests/test-S3.R:417:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L417) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                  ^
```
[inst/tests/test-S3.R:417:114:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L417) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                                                ^
```
[inst/tests/test-S3.R:424:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L424) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:424:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L424) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:424:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L424) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:426:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L426) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)), length(strsplit(res, '\n')[[1]]))
                                                      ^
```
[inst/tests/test-S3.R:428:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L428) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                  ^
```
[inst/tests/test-S3.R:428:114:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L428) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                                                ^
```
[inst/tests/test-S3.R:435:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L435) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:435:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L435) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:435:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L435) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:437:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L437) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)), length(strsplit(res, '\n')[[1]]))
                                                      ^
```
[inst/tests/test-S3.R:439:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L439) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                  ^
```
[inst/tests/test-S3.R:439:114:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L439) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                                                ^
```
[inst/tests/test-S3.R:446:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L446) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:446:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L446) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:446:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L446) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:448:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L448) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)), length(strsplit(res, '\n')[[1]]))
                                                      ^
```
[inst/tests/test-S3.R:450:51:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L450) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                  ^
```
[inst/tests/test-S3.R:450:114:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L450) *style:* **Commas should always have a space after.**
```r
               nchar(sep.cols)[1] + (length(argv[,2]) - 1) * nchar(sep.cols)[2] + nchar(sep.cols)[3] + sum(argv[,2]))
                                                                                ^
```
[inst/tests/test-S3.R:457:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L457) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:457:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L457) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:457:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L457) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:459:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L459) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)),
                                                      ^
```
[inst/tests/test-S3.R:468:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L468) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:468:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L468) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:468:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L468) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:470:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L470) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)),
                                                      ^
```
[inst/tests/test-S3.R:477:29:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L477) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                            ^
```
[inst/tests/test-S3.R:477:39:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L477) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                      ^
```
[inst/tests/test-S3.R:477:49:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L477) *style:* **Commas should always have a space after.**
```r
  res <- table.expand(argv[,1], argv[,2], argv[,3], sep.cols, style)
                                                ^
```
[inst/tests/test-S3.R:479:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L479) *style:* **Commas should always have a space after.**
```r
  expect_equal(max(sapply(strsplit(as.character(argv[,1]), '\n'), length)), length(strsplit(res, '\n')[[1]]))
                                                      ^
```
[inst/tests/test-S3.R:483:17:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L483) *style:* **Commas should always have a space after.**
```r
  cells <- c('','','')
                ^
```
[inst/tests/test-S3.R:483:20:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L483) *style:* **Commas should always have a space after.**
```r
  cells <- c('','','')
                   ^
```
[inst/tests/test-S3.R:495:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L495) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1 \\ 32',b='23')),
                                              ~^~
```
[inst/tests/test-S3.R:495:61:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L495) *style:* **Commas should always have a space after.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1 \\ 32',b='23')),
                                                            ^
```
[inst/tests/test-S3.R:495:62:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L495) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1 \\ 32',b='23')),
                                                            ~^~
```
[inst/tests/test-S3.R:497:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L497) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1 \\ 32',b='23'), justify = 'right'),
                                              ~^~
```
[inst/tests/test-S3.R:497:61:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L497) *style:* **Commas should always have a space after.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1 \\ 32',b='23'), justify = 'right'),
                                                            ^
```
[inst/tests/test-S3.R:497:62:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L497) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1 \\ 32',b='23'), justify = 'right'),
                                                            ~^~
```
[inst/tests/test-S3.R:499:48:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L499) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1',b='23')),
                                              ~^~
```
[inst/tests/test-S3.R:499:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L499) *style:* **Commas should always have a space after.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1',b='23')),
                                                      ^
```
[inst/tests/test-S3.R:499:56:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L499) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(pandoc.table.return(data.frame(a='\\1',b='23')),
                                                      ~^~
```
[inst/tests/test-S3.R:534:50:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L534) *style:* **Put spaces around all infix operators.**
```r
    res <- pandoc.table.return(x, emphasize.cells=c(3,4), plain.ascii = TRUE)
                                                ~^~
```
[inst/tests/test-S3.R:534:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L534) *style:* **Commas should always have a space after.**
```r
    res <- pandoc.table.return(x, emphasize.cells=c(3,4), plain.ascii = TRUE)
                                                      ^
```
[inst/tests/test-S3.R:538:7:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L538) *style:* **Commented code should be removed.**
```r
    # length(dim) == 1
      ^~~~~~~~~~~~~~~~
```
[inst/tests/test-S3.R:540:50:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L540) *style:* **Put spaces around all infix operators.**
```r
    res <- pandoc.table.return(x, emphasize.cells=c(3,4), plain.ascii = TRUE)
                                                ~^~
```
[inst/tests/test-S3.R:540:55:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L540) *style:* **Commas should always have a space after.**
```r
    res <- pandoc.table.return(x, emphasize.cells=c(3,4), plain.ascii = TRUE)
                                                      ^
```
[inst/tests/test-S3.R:544:7:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L544) *style:* **Commented code should be removed.**
```r
    # length(dim) > 1
      ^~~~~~~~~~~~~~~
```
[inst/tests/test-S3.R:573:1:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L573) *style:* **lines should not be more than 120 characters.**
```r
    x <- suppressWarnings(CrossTable(mtcars$cyl, mtcars$gear, expected = TRUE, resid = TRUE, sresid = TRUE, asresid = TRUE))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/tests/test-S3.R:598:26:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L598) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(type=c(1, 1, 2, 2, 3, 3), variable='n', value=c(71, 72, 68, 80, 21, 20))
                        ~^~
```
[inst/tests/test-S3.R:598:56:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L598) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(type=c(1, 1, 2, 2, 3, 3), variable='n', value=c(71, 72, 68, 80, 21, 20))
                                                      ~^~
```
[inst/tests/test-S3.R:598:67:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L598) *style:* **Put spaces around all infix operators.**
```r
    df <- data.frame(type=c(1, 1, 2, 2, 3, 3), variable='n', value=c(71, 72, 68, 80, 21, 20))
                                                                 ~^~
```
[inst/tests/test-S3.R:600:46:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L600) *style:* **Put spaces around all infix operators.**
```r
    expect_equal(pander_return(df.cast, style='simple'),
                                            ~^~
```
[inst/tests/test-S3.R:601:23:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L601) *style:* **Commas should always have a space after.**
```r
                 c('','',' type   (all) ','------ -------','  1      143  ','  2      148  ','  3      41   ',''))
                      ^
```
[inst/tests/test-S3.R:601:26:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L601) *style:* **Commas should always have a space after.**
```r
                 c('','',' type   (all) ','------ -------','  1      143  ','  2      148  ','  3      41   ',''))
                         ^
```
[inst/tests/test-S3.R:601:43:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L601) *style:* **Commas should always have a space after.**
```r
                 c('','',' type   (all) ','------ -------','  1      143  ','  2      148  ','  3      41   ',''))
                                          ^
```
[inst/tests/test-S3.R:601:60:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L601) *style:* **Commas should always have a space after.**
```r
                 c('','',' type   (all) ','------ -------','  1      143  ','  2      148  ','  3      41   ',''))
                                                           ^
```
[inst/tests/test-S3.R:601:77:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L601) *style:* **Commas should always have a space after.**
```r
                 c('','',' type   (all) ','------ -------','  1      143  ','  2      148  ','  3      41   ',''))
                                                                            ^
```
[inst/tests/test-S3.R:601:94:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L601) *style:* **Commas should always have a space after.**
```r
                 c('','',' type   (all) ','------ -------','  1      143  ','  2      148  ','  3      41   ',''))
                                                                                ^
```
[inst/tests/test-S3.R:601:111:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L601) *style:* **Commas should always have a space after.**
```r
                 c('','',' type   (all) ','------ -------','  1      143  ','  2      148  ','  3      41   ',''))
                                                                                ^
```
[inst/tests/test-S3.R:605:19:](https://github.com/Rapporter/pander/blob/200b5b839a575f9fa0a2be1c44afa0288330efa8/inst/tests/test-S3.R#L605) *style:* **Commas should always have a space after.**
```r
    ctl <- c(4.17,5.58,5.18,6.11,4.50,4.61,5.17,4.53,5.33,5.14)
                  ^
```


If the current system is too complicated for you specifically, I suggest you get someone to help you in setting it up.


yes, all `:param` descriptions have to start at the same column within one doc comment


What should  i do?


Here is still something to improve. Version of GCC is hardcoded here. I needed to do this temporary change because I had installed 4.9 and 5.3.


neat, like it!


@imcaffrey 刚刚用应用 push 的，push 前忘了 pull ，可能被 force update 了。


можно упростить, а то слишком много ветвлений. 7 условных операторов на 10 строк


It makes sense to have some checks in `Type` subclasses to handle that case, yes, but I think they should be `assert`s or should be checked at construction time. These classes cannot function reasonably if they have the wrong number of members, so the code shouldn’t try to work around those cases (using ’?’ types or whatever). It’s better to make the developer aware of the fact they’re using the `Type` subclass incorrectly, so they can fix their code, rather than returning something which, at first glance, looks valid. :-)


Fixed issues #11 #10 #3


read up on my previous review comments and find it, I won't search it up for you


元の訳からですが、「組み合わせ」は原文にありませんし、意味としておかしいので、ここは単に「それぞれの列の等価性」で良いと思います。


I've decapitated the zombie codes.


если ничего не делаете с конструктором по умолчанию, может тогда лучше положиться на компилятор?
```cpp
Tree() = default;


parentheses around the string aren't needed


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18981](https://teamcity.mesosphere.io/viewLog.html?buildId=162502&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 262 Build time: 00:10:10


@Lucifier129 Any progress on reordering? Didn't you figured it out? I just saw the benchmark results for react-lite compared with the code the person you chatted with did finish earlier. 94% performance difference!!!

Will you add support for both keyed and non-keyed or what is the idea?


transcription = CODONS[current_triplet]
    return transcription
- это должно выглядеть как-то так?
думаю, что не понимаю почему


変更部分について確認しました。


Well, I think that could be generic and useful enough to be directly
available in Message.
Best,

Federico Giaimo
PhD student in Software Engineering
Department of Computer Science and Engineering
Chalmers University of Technology
Göteborg, Sweden

On Mar 12, 2016 7:02 AM, "Christian Berger" <notifications@github.com>
wrote:

> @fgiaimo <https://github.com/fgiaimo>: Should we put this method rather
> to class Message due to its generic nature?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/se-research/OpenDaVINCI/commit/152c6bf1bcda096ea8bdc81c179960a873e54242#commitcomment-16660404>
> .
>


do you mean that i add `ask_for_action_and_apply` for inputs `1` and `echo`
and remove `1` and `echo` inputs from the above ?


ну я надеюсь, что не войдет в привычку делать такие короткие названия у типов.


> do you mean that i add ask_for_action_and_apply for inputs 1 and echo

yes

> and remove 1 and echo inputs from the above ?

no


Keine Ahnung was das hire für eine Version ist. 10 ist ja Uralt. In meiner Version passt das jetzt.


good :)


You bought a Galaxy s7?


After reading the blogpost, I see why its not good to litter codes with zombie codes. I've removed the zombie codes.


I'd set `self.config_options` to `None` here, otherwise it'll be reset anyway right?


I've removed the hash.


this test seems to fail

```
E           Stdout:
E            /tmp/tmpa7j5_xho: line 0, col 0, ES5 option is now set per default (I003)
E           /tmp/tmpa7j5_xho: line 2, col 10, Missing name in function declaration. (W025)
E           /tmp/tmpa7j5_xho: line 3, col 2, Function declarations are not invocable. Wrap the whole function invocation in parens. (E039)
E           /tmp/tmpa7j5_xho: line 3, col 3, Expected an assignment or function call and instead saw an expression. (W030)
E           /tmp/tmpa7j5_xho: line 3, col 4, Missing semicolon. (W033)
```


I've killed them all.


@sils1297 But the test-coverage is perfect


你在干嘛。。。


Done!


E porque quando vc pediu para cadastrar novo academic dava um monte de
mensagem de warning que a tag <a> não tinha namespace ou algo parecido.
Em 11 de mar de 2016 11:29 PM, "Vitor E. Silva Souza" <
notifications@github.com> escreveu:

> Por que não renderizar o globalMessages?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dwws-ufes/Marvin/commit/394bbbdfc7cdc328934df15c7aed3953eee174c9#commitcomment-16659612>
> .
>


#witterrules


This method calls a lot of  here to process the player's guess into a message and returns a string. All the methods that i mocked have been tested in isolation. That's why I mocked them here.


that doesn't mean that your tests are. http://www.infoq.com/news/2007/05/100_test_coverage


TeamCity Civic Assembly :: Build all [Build 95](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1919&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


you still have to test that the element is discarded, right?


Lies bitte mal hier, geht um das two-level-model
https://forum.fhem.de/index.php/topic,18920.msg128100.html#msg128100|The
rudolfkoenig:
...
The IODev entry (assigned by AssignIoPort) is only relevant when sending messages, and is evaluated by the **IOWrite call, which should be used in the logical device.**
...


@sils1297 So should it be somthing like
`with simulate_console_inputs(1, 'failing_input', 1, 'echo')\
                as generator, retrieve_stdout() as sio:
            OpenEditorAction.is_applicable = staticmethod(
                    lambda *args: True)
            ask_for_action_and_apply(self.log_printer,
                                     self.console_printer,
                                     section,
                                     metadata_list,
                                     action_dict,
                                     failed_actions,
                                     Result("origin", "message",
                                            diffs={testfile_path: diff}),
                                     self.file_diff_dict,
                                     file_dict)
            ask_for_action_and_apply(self.log_printer,
                                     self.console_printer,
                                     section,
                                     metadata_list,
                                     action_dict,
                                     failed_actions,
                                     Result("origin", "message",
                                            diffs={testfile_path: diff}),
                                     self.file_diff_dict,
                                     file_dict)
            self.assertEqual(generator.last_input, 1)
            self.assertIn('OpenEditorAction', failed_actions)`


something is off with indentation here


so should i add another ask_for_action_and_apply just under action_ask_under_apply ?


Do you mean the full file? You can see it clicking on View


i am not sure to understand every thing at this moment, but i am following
so trasactions is between two parts that's ok


That's a big requirements. Would it be possible to find an automatic way to merge fields even if their domain differs. For instance, we could create a new Raw domain for the merged field ?


Thanks :)


Fixed by 6a5e2e8 for v0.16 and 16a736a on top of v0.15.5.
This would be worth releasing v0.15.6 with the fix immediately. Please confirm that the fix is fine, then I'll release it as v0.15.6.


okg


mljklj


dalam hasil kali,bagi,tambahan, dan pengurangan ke algoritma pengoraman di awali dengan include,using lalu lanjutan


push不是默认没有-f吗…………


前の訳からですが、"obvious"を「はっきりとした」と訳していますが、これでは意味不明で、「明白」とか「自明」という意味ではないでしょう。（たとえば、INTEGER型のように)。


Can you explain the role of this new Symbol ?


You're very welcome. I really appreaciate people who either thank me or give encouraging words since I'm motivated to continue something. I'm mostly type where if I don't get some sort of motivation I drop/stop doing something.

Just to add something: in those fixes you should alslo have almost every event scripted in Outland, Aldor quest of socrezar in Netherstorm, Hellfire peninsula ones, shadowmoon valley too - cipher of damnation (this one caused crashes for me to till day so you should take a look at it, when sql will be provided. It occured randomly, like once per 15 tries, no idea what caused it.) Besides Outland most changes were also done to t4 raids, t5 only Tempest keep, and all of dungeons/heroics + all classic dungeons. ^^


dalam algoritma ini penilaian dari minim bilangan terkecil 2 sampai3


What happen if only one of the two fields has a None current value ?


@pgmmpk Just curious: can the exception list be simplified by leaving only offending stems / roots?


@ccx-testerbot runtests master


Thank you for the request. It has been added to the queue (id: 179). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/179) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


I also think it would be a good idea to make these package-private and implement utility methods on `ArtifactVersion`, relying on implementation details is not a good idea :)


Manchmal verstehe ich Deine Fragen nicht so recht, vielleicht hab ich das ganze auch noch nicht richtig verstanden. Wollte ja eigentlich auch kein profesioneller FHEM Modulschreiber werden. Welche Protokolliste soll ich überschreiben? In dem Modul gibt es keine, also muss ich eine definieren, oder? Die wird benötigt um den Data String zugenerieren und die message zusammenzubauen.


Ist drin.


what is the problem of this port


open brackets at right


this is const char *


Yeah. Everyone knows my name :o:


Hi Vladimir,

yes indeed, good find! If you add "swappable" to `migrations.CreateModel()` options, account migrations are indeed properly ignored in the use cases previously discussed:

```diff
diff --git a/accounts/migrations/0001_initial.py b/accounts/migrations/0001_initial.py
index ce3b1b1..5642170 100644
--- a/accounts/migrations/0001_initial.py
+++ b/accounts/migrations/0001_initial.py
@@ -31,6 +31,7 @@ class Migration(migrations.Migration):
             options={
                 'db_table': 'auth_user',
                 'verbose_name': 'user',
+                'swappable': 'AUTH_USER_MODEL',
                 'verbose_name_plural': 'users',
             },
             managers=[
```

What is also interesting is the function `django.db.migrations.autodetector.MigrationAutodetector.swappable_first_key()` that sorts potentially swappable models first when creating migrations, which considers the following:

```python
            if (
                model._meta.swappable or
                "AbstractUser" in base_names or
                "AbstractBaseUser" in base_names or
                settings.AUTH_USER_MODEL.lower() == string_version.lower()
            ):
                return ("___" + item[0], "___" + item[1])
```

But this is of course does not tackle the issue discussed previously.

Cheers,
Z.


use r_str_newf() instead of format, malloc, sprintf, free


OK très bien


bracket


Addressed #13


this the wrong


make this condition clear by `return get_python_version (magic) != NULL`


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 51](https://teamcity.labkey.org/viewLog.html?buildId=310098&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (1 new), passed: 44 Build time: 01:27:14
### Failed tests
```
org.labkey.test.tests.cds.CDSVisualizationTest.verifySubjectCounts: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.cds.CDSVisualizationTest: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifyPlotToolTips: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

```


right


i dont like that global var, but i dont see a better way to do this without messing too much the code, so lgtm


@klq oops, got it. Thanks!


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **3/13** |
| _DVB | 5/5 |
| _DVD | **0/3** |
| _DVR-MS | **0/2** |
| _General | **0/27** |
| _Hauppage | **0/3** |
| _MP4 | **0/2** |
| _NoCC | 10/10 |
| _Teletext | **6/21** |
| _WTV | **0/13** |
| _XDS | **0/34** |
| _Options | **8/87** |
| _DTVCC | **3/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/179) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


dalam bahasa pengoraman cout memakai nama orang, usia, tinggi dan hobby.


shouldnt be `length < sizeof (magic)` ?


ok than i will ee


TeamCity Civic Assembly :: Build all [Build 95](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1919&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 70, passed: 1348, ignored: 3 Build time: 00:15:10
### Failed tests
```
Venue tests: Venue type: Should Not Add Venue if Cancel button pressed: <no details avaliable>

Venue tests: Venue type: Should Add Venue: <no details avaliable>

Venue tests: Venue type: Should Add an Venue and CONTINUE: <no details avaliable>

Venue tests: Venue type: Should Not Edit a Venue: <no details avaliable>

Venue tests: Venue type: Should Edit a Venue: <no details avaliable>

Venue tests: Venue type: Should Not Delete a Venue: <no details avaliable>

Venue tests: Venue type: Should Delete an Venue: <no details avaliable>

Venue tests: Conference place type: Should Not Add Conference place if Cancel button pressed: <no details avaliable>

Venue tests: Conference place type: Should Add Conference place: <no details avaliable>

Venue tests: Conference place type: Should Add an Conference place and CONTINUE: <no details avaliable>

Venue tests: Conference place type: Should Not Edit a Conference place: <no details avaliable>

##### there are 60 more failed tests, see build details
```


I know this is following the style of Bonobo css, and so we should stick to it for the moment, but sometime we might consider getting rid of these very complex selectors and just putting classes on the objects we want to style, and simply referring to them - that would give better performance (if anyone cared), but more importantly wouldn't couple the style to the html structure so tightly.


@racaljk 这个……怎么听起来怎么好像有点不负责任似的……


Maybe we should introduce a define for inpustream api version, like we have for most other addons?


gfdgdfg


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18982](https://teamcity.mesosphere.io/viewLog.html?buildId=162506&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 262 Build time: 00:09:29


Why 0.0.1? Implies something in really very early dev state...


前からの訳ですが"inexact"を「不正確」と訳すのはちょっと誤解を招く気がします。「厳密ではない」ではいかがでしょう。


Magic numbers everywhere... ;-)


This commit is bugged, as you cast every type of event to an `xcb_create_notify_event_t`.Enter and leave events must both be casted to `xcb_enter_notify_event_t`, and map and unmap to `xcb_map_notify_event_t`.


Maybe we should introduce a define for codec api version, like we have for most other addons?


Why 0.0.1? See my other comment.


変更点について確認しました。


I really would like to see a define for the API level. I hate magic numbers.


@imcaffrey 我不是老司机啊、、、
对社交类网站，没有太多访问的欲望，还是等其他人来修复吧，毕竟更新 hosts 还是挺费时间的。


**Ignored Files - Critical**
Always ignore files/folders autogenerated by IDEs(eg .idea folder). They are not needed by other developers working on the same project and they change too frequently.


增加了提示


Why the conditional statement ? `$TARGET` should always be defined, shouldn'it ?


I didn't test this in Windows, I just used this `if` based on some examples.


嗯……
aofall说现在这个facebook的IP是个私人的反代……
你说要不要暂时把FB移除呢……


I was wondering the other day if requiring `symfony/symfony-standard` wouldn't be even better.


No tiene que ser un atributo!


Addressed #12


@imcaffrey 没得到允许的话，还是不要随便用别人的。


Hay que invocar al metodo validIndex!


I pushed a commit that should fix your issues.


Je viens d'y penser mais il y a également un soucis d'affichage avec la grande image. Celle-ci est systématiquement redimensionnée en 480x270px. J'ai donc ajouté un filtre pour optimiser l'affichage.

Le lien pointe quant à lui toujours sur l'image originale, sans redimensionnement.


als $id kann eine beliebige Ganzzahl verwendet werden.
Es kann aber keine Version kleiner 1 dargestellt werden da 023 als 23 angezeigt wird


Este atributo sobra!


Debía devolver un valor booleano!


**Code Quality - Nitpick**
These file operations should have been moved to another class. I see repeated file operations in a lot of places in your code.


Debía devolver un booleano!


確認しました。問題ないのでマージします。


**Code Quality - Critical**
This conditional statement is too complex. You can abstract it into an explaining method(method whose name explains what the method is doing)


Vladimir, one last question: What do you think about https://github.com/wq/django-swappable-models ?

I understand that `accounts.models.User` does not fit the use cases presented there, as the custom User model is already handled by Django and we won't actually provide an abstract, swappable model for djnro users to extend upon, but I still wonder if handling such issues using that API's methods would be more maintainable in the future and would thus be worth the extra dependency.

To be clear: I am not suggesting this for the present PR though.


Ist ja eigentlich auch noch ALPHA. Habe da nur mal eben das die Kopie von Somfy angepasst, das da was funktioniert. Hab das Ding ja nicht geschreiben. Sobald das ordentlich funktioniert, gibts die 1 davor. Von mir aus auch gleich ;-)


Solo se permite un return!!


等到有需求的人update一下吧……
不删了不能心安啊


Kannst Du mir das bitte genauer erklären? Ich verstehe das nicht. Weiter unten habe ich die generierung des raw codes implementiert. Da kann ich 000b und 15 eigentlich nicht gebrauchen. Es sei denn cih schreibe alles nochmal neu.


Esta llamada debe ir dentro de un if!


**Code Quality - Nitpick**
I wonder why you created a module called SimpleMethods. This module can be refactored to a class called `Integrity` since most of the methods here deals with `integrity of something`.


Nombre de la clase la primera en mayúscula!


Esto no se pide!


Nombre del archivo debe ser igual a la clase!


No es necesario sacar ningun mensaje!


Penalizado por la existencia de un solo commit!


Om Api-wrappern inte fungerar som den ska så är detta commiten att titta på. Testade lite och det verkade funka för mig.


**Test - Nitpick**
I am not sure what is been tested here. All the expectations returned mocked results. Why is that?


Kasneje sem ugotovil da je to narobe in tudi spremenil. Moralo bi pisati tako :
style="width: 200px;style="text-align:right; "


Solo se permite un unico return!


Yeeeaah!


internal storage? it "get**External**CacheDir"


All good. Again thank you for the help
On 12 Mar 2016 01:42, "Mike Alexandersen" <notifications@github.com> wrote:

> Sorry about this. I think the extra space before the if's and elifs must
> have been some sort of copy-paste issue, as it looks right in my file on my
> computer. The lower number of indent for the cx lines is clearly because I
> forgot to check how many spaces you use (I tend to prefer two spaces per
> level and clearly just used that without thinking). Sorry.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fraserw/trippy/commit/2c0a8a3f45dc17faf8f476ad355c63d8825fbc55#commitcomment-16660543>
> .
>


Looks good and I appreciate you adding some tests too!

If you can sort out the mismatched line endings (resulting in so many lines changed), I'll get it in.


**Test**
Is all these mocking necessary to test a particular function? Also why are all the mocked methods returning nil?


**Atomic commit - Critical**
This comment is doing too much. Next time you commit these way again, I will tell you to restart the project. Make sure your commits are atomic.


Este calculo deberia ir en el constructor!


Why are you using specify here?


Este calculo está mal hecho!


Kann es sein das da ein paar * fehlen? Sollte das nicht so sein:
$hash->{AutoCreate} =
      { "dooya.*" => { ATTR => "event-min-interval:.*:300 event-on-change-reading:.*",
                       FILTER => "%NAME",
                       autocreateThreshold => "2:10"


@ccx-testerbot runtests master


Even een goeie error message zetten hè.


Thank you for the request. It has been added to the queue (id: 180). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/180) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


@kkaefer shouldn't the condition be reversed? --silent when non-interactive, and spamming progress otherwise?


The proposed patch to fix missing superblock readout.
[fix-sb-reading.txt](https://github.com/jxiong/zfs/files/170367/fix-sb-reading.txt)


@romain-chiap Can you explain what you were doing by adding new files outside of the `PythonWorkspace` folder?  There now seems to be duplicates of `base_robot.py` for example.  Great work though, just trying to get a handle of which files I should be using in the future!


Misspelling problem


首页


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **3/13** |
| _DVB | 5/5 |
| _DVD | **0/3** |
| _DVR-MS | **0/2** |
| _General | **0/27** |
| _Hauppage | **0/3** |
| _MP4 | **0/2** |
| _NoCC | 10/10 |
| _Teletext | **6/21** |
| _WTV | **0/13** |
| _XDS | **0/34** |
| _Options | **8/87** |
| _DTVCC | **3/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/180) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


Changes are included
1.Changed setFile method variable name.
2.Changed saveContent method taken length variable


Woot! you should enable the obj loader in the robotics modid, so that people can use objs.


Thank you for the request. It has been added to the queue (id: 181). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/181) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


That's correct. I'm agree


제목을 왜 이따구로 했나


내가 혼자 테스트


Not really a good idea the way things are right now, I'm afraid.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18983](https://teamcity.mesosphere.io/viewLog.html?buildId=162510&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 254 Build time: 00:05:22


По букингу,немного пофикситт стили кнопок, убрать рамку у них
И графа пожеланий , почему только олна строчка ?
На вторую строку не переходит


Contributes to #3


your test doesn't test that this works! It has the same invalid files as the test in line 41, your test should show that the settings *change* the outcome of the analysis


@imcaffrey 算了，我尝试更新下，不过目前的规则列表好像不完整，缺失的我就不管了。


why not define the arguments outside the function?


pointer to the issue would be nice


По менюшеке,ты сделал черным? Или баг...


As long as we don't copy paste code to provide that append functionality you can remove that too.


GitHub release title says 4.02.3. Which is it?


Penalizado por ser un unico commit!


Yes. I simply don't want to reuse ignore after is clear how I want to use it. Replace it then.


AFAICT this solves mholt/caddy#549 more elegantly than by patching https.RequestCallback and providing global flags like I did.


Sin archivo gitignore!