@ShMcK Hey, it looks like your edits of this step broke the tutorial. It doesn't run now; hope Urigo hasn't deployed it yet. Then, some of the edits broke the mark-up: broken list in the introduction, why do you use -- everywhere instead of —? Also, I don't why you've added a paragraph about extending the user type and adding it to `party.d.ts`. We don't touch the user type in this step, at least we don't use this "invited" property (which is of some strange type `string[]`) in any place at this point. So people will be likely confused, like me.
You've create an issue for this. So why not to discuss it and then add in here?
Can you be, in general, more careful with edits? It's the only step I looked into after the edits. I hope other steps are still ok.


replaced


I recommend using the Google Fonts Developed  API, but if you wish to
continue using metadata files, you can use your own branch of the repo with
those files from before this commit while switching to protocol buffers :)


` $buffer_related .= "
View Spring Graph of Variant Characters | Variant characters of variant characters will also be shown.
"`


lose } in cm.dependencies


WeakRef!!!


my local is fixed...


Why the `license.rtf`? If it don't state otherwise all the content of the repo is already GPL3, and there is already a `LICENSE` file in the root of the repo.


` $buffer .= qq|<div class="texts"><a href="/wiki/Special:Renewall?view=listup&target=$wiki_name">Batch Replace Components of Existing Glyphs</a></div>|;`


Thanks :smile:


aaaaaaaaaaaaaaaaaaa


is it ok if we still use globals like `$GLOBALS['wc_customizer'] ` for free plugins?
(customizer does that in main class and settings)


What;s wrong with CMAKE...

LINK_INTERFACE_LIBRARIES
INTERFACE_LINK_LIBRARIES

and both are supposed to be valid and do different things - those almost identical names are certainly a great starting point for confusion....


I know. I just don't think this case is comparable, though, and I'd rather stick to the common coding style of the file.


`$buffer .= qq|<h2>Pages that Cite this Glyph</h2>|;`


Would this still work if you used `controllerAs: 'exampleCtrl'` or even 'vm' for all of them? If so, I think that is more clear to people reading the code. Otherwise I could see how a viewer might think that the SceneViewCtrl is somehow related to the scene view directive.


Since we can't use lamdas and we don't have a functional interface with this purpose (yet), it was the only possibility. And I am not even sure whether the functional interfaces would work.


You mean something like this?
```
       HOST=LINUX
-      LIBGL="-lGL -lGLU"
+      PKG_CHECK_MODULES([GL], [glu glew])
+      LIBGL=$GL_LIBS
       ;;
```
Actually I now checked and this patch could be dropped, it compiles fine without it. The question is - do we need the pkg-config call or we can leave in place the hardcoded -l flags from upstream.


smart :) :+1:


ok, see https://github.com/linagora/james-project/pull/83


>>Maybe something like optional "drawSelectionFunction" parameter for init method? If it is not set - we'll use default function.

Styles or function?


An import of terminal (console?) was failing.
On Jan 12, 2016 2:22 AM, "Peter Magnusson" <notifications@github.com> wrote:

> what didn't work with 3.0?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/romanchyla/nodemcu-uploader/commit/37004fad7758653be3c465a668570009e959b6b9#commitcomment-15389886>
> .
>


hehe no problem :-)

thx for the link! - Sounds interesting ! .... but strange that it breaks wifi.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1213.d091cf87717b8347142f1458231711deeed5e6cf](http://build.palaso.org/viewLog.html?buildId=67866&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 824, ignored: 7 Build time: 00:21:09


` $buffer .= qq|<h2>Edit Talk Page</h2>|;`


Exciting?


throw Throwable.propagate(e)


unit-hue-3[4047]: 2016-01-12 15:40:05 INFO unit.hue/3.install logger.go:40   File "lib/charms/bootstrap.py", line 31, in install_charm_deps
unit-hue-3[4047]: 2016-01-12 15:40:05 INFO unit.hue/3.install logger.go:40     cfg = layer.options('basic')
unit-hue-3[4047]: 2016-01-12 15:40:05 INFO unit.hue/3.install logger.go:40 AttributeError: 'module' object has no attribute 'options'


` <a name="metaedit"><h2>Edit Metadata</h2></a>`


Hey! At least i fixed the first mistake


It looks like the target `.build/dev-requirements.timestamp` is never declared. Did I miss something?


Hi Jim, one doubt: Do you not need update this line too?

openvpn-client-export.inc : 604
exec("/bin/cat {$tempdir}/7zS.sfx {$tempdir}/7zipConfig {$tempdir}/archive.7z > {$outfile}");

with the new name sfx: 7zsd_All.sfx


@Mumfrey It wasn't because of a fork, it was due to me overwriting my fork's history after SpongeVanilla's history rewrite. :innocent:


I just use same vimrc with vim and neovim ,and add alias vim='nvim' into bashrc,then I can use
vim Foo,java    open by neovim
vi Foo.java       open by vim
and here is my vimrc
https://github.com/wsdjeg/DotFiles


I'm not fluent in CMake and was even more confused after reading cmake --help-policy CMP0022 ...


Esto sobrescribe el bloque de skills de ```if (api_get_setting('allow_skills_tool') == 'true') {``` de la línea 366
Los dos bloques deben ser $blocks['skills']?


add empty line at the end


It is failing on a lot systems. This is not the way to force it off if enabled.
Example, my main system uses PHP 7.  Imagicalmine uses PHP5 (~/path/to/imagicalmine/bin/php5/)
So don't even mess with such commands!


@monsta
rebased and used for both toolkit versions


private ?


Thanks, fixed.

While I have you on the hook, could you fix this warning in the header `bundle.hpp`?

```
./bundle/bundle.hpp:164:22: warning: unused variable 'h' [-Wunused-variable]
            unsigned h = input[pad + 0];
```

It's not that critical, which is why I didn't open an issue. I know that compiling the source emits tons of warnings and they are hard to avoid, but IMHO the user-facing header should aim for higher standards.


if an error occurred, there's no sense continuing on, so an early `return` should be used.

also, when logging errors, it's good practice to use `console.error` as it will print to stderr instead of stdout.

```
return console.error(error);
```

or better yet, set the exit code to 1 since that signals an error in the unix world:

```
console.error(error);
process.exit(1);
```


I think it's not AuthenticationFilter responsibility.


you have to specify a Locale for toLowercase


flops.


could be `else if`


sorry, I meant .ifPresent()


I just gave it a try too... it really does not help :)


same note about error logging and exit codes as above


use exception in to break the main flow instead of conditions


Why aren't you on Gitter? BTW, did you see the huge update?


It lets you do that because it's a warning (for primitive types only?) not an error.


seems like this was never used?


@yehudasa when Adam's c++11 concurrency patchset (https://github.com/ceph/ceph/pull/5931) was being discussed initially, it was agreed that we would start using the standard library types in new code, and gradually convert the rest of the codebase. so while we're losing some consistency with the rest of rgw, we'd really prefer to keep moving forward with this


` $buffer .= "<div class=\"warning2\">Error: glyph source data contains an alias referring to a non-existent glyph. The glyph cannot be rendered.</div><hr>";`


Sounds good. I just feel like there may be away to address validation errors in one test, or at least combine validation errors on the same field, since we have the same return value. Or maybe, we could update the return value to be more clear about what is causing an error.


111411352_ATDeprecateFields (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2086 of 2086 tests passed, 0 test warnings)

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


getMapKeyType and getMapValueType then? That seems more clear to me.


Two points here:
 - You should be using `TGT` here instead of `MAVLINK_HEADERS`, otherwise, 'mavgen' will always have the same output. That would be awkward from a task perspective.
 - I think it would be more portable if we called `python` directly here instead of just trusting that the system will be able to run python for us. So I think something like `${PYTHON} ${MAVGEN} (...)` would be a better choice.


I'm getting error messages after this commit during installation

```
Preparing to unpack shim_15.7_amd64.deb ...
Unpacking shim (15.7) ...
Setting up shim (15.7) ...
Installing new version of config file /etc/init.d/shimsvc ...
-su: iquery: command not found
```


Ok, i will add here GTK version checks.


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


oh wait i think i see


Idkw hat happened to the message.
We have <key> and <value> colours.


dette er kun på vilje (men det vet du kanskje). kongull er litt annerledes hvis den skal brukes.


a date library like `moment.js` would help make this much more concise, but I usually use `Date::toString` or `Date::toISOString` until I need something more humanized.


TypeEntry would work, but not be a clean solution. Think about it:
We would use a TypeSet in a Property, but have it convert one string into two, then these get converted by their inner readers. If this is a very simple thing, such as a players name and a number this might work, but as soon as the inner readers of the TypeEntry get really fance (such as a PS or similar tokenizing reader), we really get into troubles.


hadoop-client is conflicting with this change.  We need to update it to use this interface and drop its copy of `lib/charms/layer.py`


Or we could traverse level-order, right? I thought about this, but convinced myself later that pre-order is also fine. :smile: Level-oder would ensure that *any* parent is up-to-date, at any time.


the documentation seems to be swapped? pin_raw is used like device/channel pair.


it's not an iterator


add that as an option? do you see this changing?


It's a very small cost in time. I'd like to see both there.


Hi,
I wonder if it's possible to have this commit in a 3.4.3 ?

many thanks

regards

Pierre


Thanks @Crayon2000. I'm not sure what I am seeing here:

```
+#if !defined(__BORLANDC__)
 #if MIN_KEYLENGTH > 0
 		if (keylength < (size_t)MIN_KEYLENGTH)
 			return MIN_KEYLENGTH;
 		else
 #endif
+#endif
 		if (keylength > (size_t)MAX_KEYLENGTH)
 			return (size_t)MAX_KEYLENGTH;
 		else
```

Does Borland have a problem with `MIN_KEYLENGTH > 0`? My apologies if we already talked about it.


is the `.hasMoreElements()` really needed ?


Hi! I just wrote an intro.
A lot of data preservation will rely on github, yes. So eventually I think I will organise everything you will have written in a way that is not too redundant. But for the moment I'd like to let everyone write something by themselves to avoid closing doors to some of you.

As for very precise information about preservation of data etc, it can wait for the second version.


Can you please add an example on the use of `:filter`? Thanks!


Use  `six.moves.range` to support Py3.


Nope, this is a copy/paste from pwm.json


Use six.moves.range to support Py3.


well there might be users who want kodi to use their common /home/username/.ssh dir maybe - no? Its not sure if UserHome is always set to /home/username on posix imo...


sometimes you put a space before the opening brace on functions, sometimes you don't.  I favor having them, but either way make sure the style is consistent throughout.


refer to https://devtalk.nvidia.com/default/topic/490810/really-slow-cudagetdevicecount-several-seconds-to-complete-a-cudagetdevicecount-call/ ; when running gunrock and nvidia-smi loop in background, the cudaGetDeviceCount() is still very slow, but nvidia-smi itself becomes super fast (normally it runs ~10sec each time) when there is process running on any GPU.


AddNetAreaColumnToEnvelopeSummary (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2051 of 2051 tests passed, 0 test warnings)

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


No i am currently in spanish class. I also cant go on until probably around 7 because of lots of stuff tonight


Gitter doesnt work at my school


Why not call just call
  this(other, false, selection);
?


nullity should not be silently ignored, see documentation : `If the container does not allow access to header information, return null`


the contents of an anonymous namespace are only visible within the compilation unit, exactly as if declared static. but i can switch to using static instead, if you prefer it that way


111411352_ATDeprecateFields (Nigusse) - 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)


No, this test only works with this commit (preferring the parentItem adds support for reparenting).


why is that? Can't you just use the irange's min/max/step to map it? (also document min/max is being used, or 0-100 is used... but document it)


Did you find any other defects besides the bug I created an issue about?

I'll continue working with the implementation, but at the moment I am only aware of the problem with crashes after a certain amount of keyboard keys have been pressed. I think this is related to not mapping the device to a port causing some sort of overflow.


Alternatively, let's use instead of `'sceneViewExampleCtrl`' instead of `'SceneViewCtrl'`- that's the most clear, and it takes advantage of the trouble that you've already gone to to make the controller names distinct and configurable.


Agree that we need to check for `null` here.


Why would it create an optional value ?


I want to keep the gui open so that we can see the error.  We may want to add a function to robot to stop the robot


You could consider dropping PHP 5.3 support in general, and switching to short array syntax would be one benefit of that. I'm sure others too :)


TeamCity pull requests :: netty [Build 1017](https://garage.netty.io/teamcity/viewLog.html?buildId=4700&buildTypeId=pulls_netty&guest=1) is now running


I was just about to add this too! Are you actively working on this project?


@svost проверил на linux. Консольный клиент точно так же вылетает, а qt работает.
странно. ведь rpc и там и там одинаково работает :(


I will in fact be able to be on rhe wikia chat until not too long befor 3:00 because my school doesnt block it


As Adam pointed out, you shouldn't need the `sudo` here since you've switched the path to `~/.gu/`


@ralphtheninja that's what the cool kids these days use :P


Ah, okay


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


*Kuma, what have you done?


You can just return the promise. Evaluating the promise's `.then` callback and returning that value will return the same value as this.

```js
return this.rootElement.$('.displayValue').getAttribute('value');
```


Updated the daily status tracker.


Now, line 81 exceeds line-length. Or doesn't it anymore?? Anyway, that was the reason for removing the whitespace. Doesn't clang format correct this?


I would not classify my involvement as "active" ...

On 12 January 2016 at 16:56, Brian Detweiler <notifications@github.com>
wrote:

> I was just about to add this too! Are you actively working on this
> project?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/igniterealtime/Fastpath-webchat/commit/c7cca4e0fda04a98acbdcf39dd236107af37dc01#commitcomment-15400723>
> .
>


@mojodna no, this wasn't raising issues but I wasn't sure that an empty array would result in a noop. If you're confident it does, then I have no problem writing this either way.


All good points.  I was just going to mention that in a new commit https://github.com/Esri/angular-esri-map/commit/562380416ff40ce2acc8a14b667c2c80c2cc3ca4 I have  working example of using vm in the router and in the example html templates.  However, there is still the (arguably) confusing difference of having a distinct & different controller name in the associated example js files.  Thoughts?

In your last comment, are you thinking of keeping the unique controller names for the sake of the snippets?


Hmm -- well, I don't think there's much point in just testing that data gets imported.


TeamCity pull requests :: netty [Build 1018](https://garage.netty.io/teamcity/viewLog.html?buildId=4701&buildTypeId=pulls_netty&guest=1) is now running


Hoegaarden papa 😂 🙌


lovely


AddNetAreaColumnToEnvelopeSummary (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1502 of 1502 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.67%25-green.svg)


I was following a different approach: any test added together with (production) code, needs that code to become green.


There is no jquery reference in bower.json file.


How about using `testing.parameterize` to test three dimensional and two dimensional cases. It enables us to unify `test_***` and `test_***_three_dim`.

One of the possible implementation is:
```
@testing.parameterize{
    {'shape': (4, 3, 5)},
    {'shape': (4, 15)},
}
class TestL2NormSquared(unittest.TestCase):
     def setUp(self):
          self.x = numpy.random.uniform(-1, 1, self.shape).astype(numpy.float32)
          self.gy = numpy.random.uniform(-1, 1, self.shape[0])
    ...
    def test_forward_cpu(self):
        self.check_forward(self.x)
```


https://github.com/juju-solutions/layer-hadoop-client/pull/1
https://github.com/juju-solutions/layer-hadoop-base/pull/3


You're right. I guess I was trying to figure out what it does in all cases and just jotted it out there -- this is logic extracted from the functions where it was duplicated in.


See other comment.


Yeah, pressing and holding volume up/down could crash RA.


@monsta
rebased and and gtk version checks are used


I think it's an useful idiom/pun. "DRY" is a well-known acronym for "Don't Repeat Yourself", which this is doing, and when making code more DRY... you DRY it out. :)


辛苦 zz 菊苣，

评论框底部的线可以不用：
![image](https://cloud.githubusercontent.com/assets/1846356/12268603/72b6bd2c-b988-11e5-866e-764fa6912bab.png)

对于种子页面的标题希望调整成和列表的标题样式一致，滚动条覆盖在标题栏之上，半透明。
另外对于种子和列表页里的语言（tag、团队名）能否调整成全局的语言？
![image](https://cloud.githubusercontent.com/assets/1846356/12268675/b7e47506-b988-11e5-91fd-640dda8e1120.png)

对于右侧的按钮：
![image](https://cloud.githubusercontent.com/assets/1846356/12268699/cf36ae72-b988-11e5-9a25-87f16c226e2e.png)
希望能放在原来的几个按钮之下，种子页面加载完成后以从下到上的动画出现~


yes, that is correct. bootstrap-sass defines jquery as a dependency in their bower.json file.

https://github.com/twbs/bootstrap-sass/blob/d07b75677e830090e11a0039613e9bdc5ba65df1/bower.json#L33


good call. Thanks.


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


Sorry to bug you -- can you push bumped version to Hex, please? Thank you!


Works fine for me. I opened one of the chests in Candlekeep, pressed X for the menu to appear and normally saved. I was properly dropped to the main menu. Default resolution, in case the window pack will matter. Did you do something differently?


test


This way 'cd_copyrel' becomes the default make target instead of 'all'


wow, there's a lot of magic string matching in this code. outside the scope of this PR, but this area could use some love.


I removed the space before the `//` instead of behind :)


@monsta
yes, i thought the same


This causes the migrations to fail if run on a new system (ie the test suite) as your other migration then attempts to remove a column which doesn't exist.

It's usually best practice to not change old migrations for this reason.


Thanks! I will analyze this approach :+1:


The scripts directory is not a package so the import fails here. I don't think it needs to be a package though, because we shouldn't import it anywhere. So we could manipulate `sys.path` here.

It might easier to just make it a Python package though. Happy to reconsider if it's too unreliable.


The project is missing the PHP backend + The Project is made with Unity < 4.3


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 131](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=659&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


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


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 131](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=659&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:19


Hello, 'super.webView' is instance of CordovaWebView and it haven't getHandler() method.
Could you fix it for test your last commits, please?


Ah! Missed thar! Sorry for the noise :smile:


위 두변수를 사용하는데가 없는것으로 보입니다.
확인 바랍니다.


Mmh, I guess I kind-of described an implementation level-order traversal in my comment :flushed:


> @ralphtheninja that's what the cool kids these days use :P

It's not easy to be an old fart like me and try to catch up with the new cool things :grinning:


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


unused ?


missed that one as I was always deleting bootstrap.


> This is for the case you want to force the rendering before serialization. You can either call render() or pass render=True (the short variant).

But `render=True` *doesn't* force rendering in all cases, i.e. it behaves differently from calling `render()`! This is important to note.


Nope, because for inline TileJSON, there's no way how it can be a "Mapbox source".


you should remove the generic, it only adds complexity


public ?


sp direction


Ideally a regular user should never see this:

* These names only exist when a job-in-job job is running
* They are passed from the api to the front end, and ideally the front end will parse these in JS to display the correct name, and also provide a view of the sub-jobs using the information.
* The run-logs will be saved with this name, so they will be in mongo, but when pulling logs to the front end, these should also be parsed in JS before being displayed to the user
* Lastly is emails, same idea, the email template should parse out the delimiter

In the tests, I manually change the delimiter to a pipe `|` so it is human readable, but the intention is for the delimiter to be only machine-readable


use a submethod instead of a comment


nit: couldn't this just be `todo.erase(todo.begin())` to save the map lookup?


In general, view helpers should not depend on this for anything, but rather be purely functional. The big advantage in Derby is reactivity (which you won't get if you use this.XXX to grab something inside of your view helper). This might not be an issue here, but for good practice that might be a better way to write your view helpers.


same comment


test


"*are* always named... *and are*"?


No, this is wrong, it should be replaced by `.build/python-venv/bin/tx` :-)


`throw Trowables.propagate(e)`


[R/exa.createScript.R:101:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L101) *style:* **Variable and function names should be all lowercase.**
```r
exa.createScript <- function(channel, name, func = NA,
^~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:103:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L103) *style:* **Variable and function names should be all lowercase.**
```r
                             initCode = NA,
                             ^~~~~~~~
```
[R/exa.createScript.R:104:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L104) *style:* **Variable and function names should be all lowercase.**
```r
                             cleanCode = NA,
                             ^~~~~~~~~
```
[R/exa.createScript.R:105:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L105) *style:* **Variable and function names should be all lowercase.**
```r
                             inType = SET,
                             ^~~~~~
```
[R/exa.createScript.R:106:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L106) *style:* **Variable and function names should be all lowercase.**
```r
                             inArgs = list(),
                             ^~~~~~
```
[R/exa.createScript.R:107:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L107) *style:* **Variable and function names should be all lowercase.**
```r
                             outType = EMITS,
                             ^~~~~~~
```
[R/exa.createScript.R:108:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L108) *style:* **Variable and function names should be all lowercase.**
```r
                             outArgs = list(),
                             ^~~~~~~
```
[R/exa.createScript.R:109:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L109) *style:* **Variable and function names should be all lowercase.**
```r
                             outputAddress = NA,
                             ^~~~~~~~~~~~~
```
[R/exa.createScript.R:110:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L110) *style:* **Variable and function names should be all lowercase.**
```r
                             replaceIfExists = TRUE,
                             ^~~~~~~~~~~~~~~
```
[R/exa.createScript.R:111:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L111) *style:* **Variable and function names should be all lowercase.**
```r
                             mockOnly = FALSE
                             ^~~~~~~~
```
[R/exa.createScript.R:115:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L115) *style:* **Variable and function names should be all lowercase.**
```r
  initCode <- m$initCode
  ^~~~~~~~
```
[R/exa.createScript.R:115:17:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L115) *style:* **Variable and function names should be all lowercase.**
```r
  initCode <- m$initCode
                ^~~~~~~~
```
[R/exa.createScript.R:116:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L116) *style:* **Variable and function names should be all lowercase.**
```r
  cleanCode <- m$cleanCode
  ^~~~~~~~~
```
[R/exa.createScript.R:116:18:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L116) *style:* **Variable and function names should be all lowercase.**
```r
  cleanCode <- m$cleanCode
                 ^~~~~~~~~
```
[R/exa.createScript.R:118:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L118) *style:* **Variable and function names should be all lowercase.**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
  ^~~~~~
```
[R/exa.createScript.R:118:23:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L118) *style:* **Variable and function names should be all lowercase.**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
                      ^~~~~~
```
[R/exa.createScript.R:118:31:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L118) *warning:* **no visible binding for global variable ‘ALLOWED_UDF_IN_TYPES’**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
                              ^~~~~~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:119:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L119) *style:* **Variable and function names should be all lowercase.**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
  ^~~~~~~
```
[R/exa.createScript.R:119:24:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L119) *style:* **Variable and function names should be all lowercase.**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
                       ^~~~~~~
```
[R/exa.createScript.R:119:33:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L119) *warning:* **no visible binding for global variable ‘ALLOWED_UDF_OUT_TYPES’**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
                                ^~~~~~~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:121:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L121) *style:* **Variable and function names should be all lowercase.**
```r
  inArgs <- do.call(paste, c(as.list(inArgs), sep = ", "))
  ^~~~~~
```
[R/exa.createScript.R:121:38:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L121) *style:* **Variable and function names should be all lowercase.**
```r
  inArgs <- do.call(paste, c(as.list(inArgs), sep = ", "))
                                     ^~~~~~
```
[R/exa.createScript.R:123:7:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L123) *style:* **Variable and function names should be all lowercase.**
```r
  if (outType == EMITS) {
      ^~~~~~~
```
[R/exa.createScript.R:124:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L124) *style:* **Variable and function names should be all lowercase.**
```r
    if (is.null(m$outArgs)) {
                  ^~~~~~~
```
[R/exa.createScript.R:127:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L127) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- paste("(",
    ^~~~~~~
```
[R/exa.createScript.R:128:47:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L128) *style:* **Variable and function names should be all lowercase.**
```r
                     do.call(paste, c(as.list(outArgs), sep = ", ")),
                                              ^~~~~~~
```
[R/exa.createScript.R:131:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L131) *style:* **Variable and function names should be all lowercase.**
```r
    outType <- RETURNS
    ^~~~~~~
```
[R/exa.createScript.R:131:16:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L131) *warning:* **no visible binding for global variable ‘RETURNS’**
```r
    outType <- RETURNS
               ^~~~~~~
```
[R/exa.createScript.R:132:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L132) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- as.character(outArgs)
    ^~~~~~~
```
[R/exa.createScript.R:132:29:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L132) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- as.character(outArgs)
                            ^~~~~~~
```
[R/exa.createScript.R:135:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L135) *style:* **Variable and function names should be all lowercase.**
```r
  sql <- paste("CREATE", if (replaceIfExists) "OR REPLACE" else "", "R",
                             ^~~~~~~~~~~~~~~
```
[R/exa.createScript.R:136:16:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L136) *style:* **Variable and function names should be all lowercase.**
```r
               inType, "SCRIPT", name,
               ^~~~~~
```
[R/exa.createScript.R:137:21:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L137) *style:* **Variable and function names should be all lowercase.**
```r
               "(", inArgs, ")",
                    ^~~~~~
```
[R/exa.createScript.R:138:16:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L138) *style:* **Variable and function names should be all lowercase.**
```r
               outType, outArgs, "AS")
               ^~~~~~~
```
[R/exa.createScript.R:138:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L138) *style:* **Variable and function names should be all lowercase.**
```r
               outType, outArgs, "AS")
                        ^~~~~~~
```
[R/exa.createScript.R:140:18:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L140) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(m$outputAddress))
                 ^~~~~~~~~~~~~
```
[R/exa.createScript.R:144:24:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L144) *style:* **Variable and function names should be all lowercase.**
```r
                       outputAddress[[1]], "', ", outputAddress[[2]], ")",
                       ^~~~~~~~~~~~~
```
[R/exa.createScript.R:144:51:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L144) *style:* **Variable and function names should be all lowercase.**
```r
                       outputAddress[[1]], "', ", outputAddress[[2]], ")",
                                                  ^~~~~~~~~~~~~
```
[R/exa.createScript.R:157:16:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L157) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(initCode)) {
               ^~~~~~~~
```
[R/exa.createScript.R:159:10:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L159) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(initCode)) {
         ^~~~~~~~
```
[R/exa.createScript.R:159:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L159) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(initCode)) {
                             ^~~~~~~~
```
[R/exa.createScript.R:160:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L160) *style:* **Variable and function names should be all lowercase.**
```r
      sql <- paste(sql, codeLine, sep = "\n")
                        ^~~~~~~~
```
[R/exa.createScript.R:165:16:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L165) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(cleanCode)) {
               ^~~~~~~~~
```
[R/exa.createScript.R:167:10:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L167) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(cleanCode)) {
         ^~~~~~~~
```
[R/exa.createScript.R:167:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L167) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(cleanCode)) {
                             ^~~~~~~~~
```
[R/exa.createScript.R:168:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L168) *style:* **Variable and function names should be all lowercase.**
```r
      sql <- paste(sql, codeLine, sep = "\n")
                        ^~~~~~~~
```
[R/exa.createScript.R:173:8:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L173) *style:* **Variable and function names should be all lowercase.**
```r
  for (codeLine in deparse(code)) {
       ^~~~~~~~
```
[R/exa.createScript.R:174:23:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L174) *style:* **Variable and function names should be all lowercase.**
```r
    sql <- paste(sql, codeLine, sep = "\n")
                      ^~~~~~~~
```
[R/exa.createScript.R:178:8:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L178) *style:* **Variable and function names should be all lowercase.**
```r
  if (!mockOnly) {
       ^~~~~~~~
```
[R/exa.createScript.R:179:9:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L179) *style:* **Variable and function names should be all lowercase.**
```r
    if (odbcQuery(channel, sql) == -1) {
        ^~~~~~~~~
```
[R/exa.createScript.R:180:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L180) *style:* **Variable and function names should be all lowercase.**
```r
      stop(odbcGetErrMsg(channel)[[1]])
           ^~~~~~~~~~~~~
```
[R/exa.createScript.R:180:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L180) *warning:* **no visible global function definition for ‘odbcGetErrMsg’**
```r
      stop(odbcGetErrMsg(channel)[[1]])
           ^~~~~~~~~~~~~
```
[R/exa.createScript.R:185:36:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L185) *style:* **Variable and function names should be all lowercase.**
```r
  function(..., table, where = "", groupBy = "", restQuery = "",
                                   ^~~~~~~
```
[R/exa.createScript.R:185:50:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L185) *style:* **Variable and function names should be all lowercase.**
```r
  function(..., table, where = "", groupBy = "", restQuery = "",
                                                 ^~~~~~~~~
```
[R/exa.createScript.R:186:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L186) *style:* **Variable and function names should be all lowercase.**
```r
           returnSQL = FALSE, reader = NA, server = NA) {
           ^~~~~~~~~
```
[R/exa.createScript.R:189:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L189) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(table))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:190:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L190) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(where) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:192:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L192) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(where))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:193:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L193) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(where) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:195:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L195) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(groupBy))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:195:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L195) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(is.character(groupBy))
                             ^~~~~~~
```
[R/exa.createScript.R:196:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L196) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(groupBy) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:196:24:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L196) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(length(groupBy) == 1)
                       ^~~~~~~
```
[R/exa.createScript.R:198:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L198) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(restQuery))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:198:30:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L198) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(is.character(restQuery))
                             ^~~~~~~~~
```
[R/exa.createScript.R:199:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L199) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(restQuery) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:199:24:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L199) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(length(restQuery) == 1)
                       ^~~~~~~~~
```
[R/exa.createScript.R:213:9:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L213) *style:* **Variable and function names should be all lowercase.**
```r
    if (groupBy != "") {
        ^~~~~~~
```
[R/exa.createScript.R:214:7:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L214) *style:* **Variable and function names should be all lowercase.**
```r
      groupBy <- paste(" GROUP BY", groupBy, sep = " ")
      ^~~~~~~
```
[R/exa.createScript.R:214:37:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L214) *style:* **Variable and function names should be all lowercase.**
```r
      groupBy <- paste(" GROUP BY", groupBy, sep = " ")
                                    ^~~~~~~
```
[R/exa.createScript.R:218:32:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L218) *style:* **Variable and function names should be all lowercase.**
```r
                 table, where, groupBy, sep = "")
                               ^~~~~~~
```
[R/exa.createScript.R:219:47:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L219) *style:* **Variable and function names should be all lowercase.**
```r
    sql <- paste("SELECT * FROM (", sql, ")", restQuery, sep = "")
                                              ^~~~~~~~~
```
[R/exa.createScript.R:221:9:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L221) *style:* **Variable and function names should be all lowercase.**
```r
    if (returnSQL) {
        ^~~~~~~~~
```
[R/exa.createScript.R:224:7:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L224) *style:* **Variable and function names should be all lowercase.**
```r
      execArgs <- list(channel, sql)
      ^~~~~~~~
```
[R/exa.createScript.R:226:9:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L226) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, reader = reader)
        ^~~~~~~~
```
[R/exa.createScript.R:226:23:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L226) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, reader = reader)
                      ^~~~~~~~
```
[R/exa.createScript.R:229:9:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L229) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, server = server)
        ^~~~~~~~
```
[R/exa.createScript.R:229:23:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L229) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, server = server)
                      ^~~~~~~~
```
[R/exa.createScript.R:231:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L231) *style:* **Variable and function names should be all lowercase.**
```r
      if (!mockOnly) {
           ^~~~~~~~
```
[R/exa.createScript.R:232:17:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L232) *style:* **Variable and function names should be all lowercase.**
```r
        do.call(exa.readData, execArgs)
                ^~~~~~~~~~~~
```
[R/exa.createScript.R:232:17:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L232) *warning:* **no visible binding for global variable ‘exa.readData’**
```r
        do.call(exa.readData, execArgs)
                ^~~~~~~~~~~~
```
[R/exa.createScript.R:232:31:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/exa.createScript.R#L232) *style:* **Variable and function names should be all lowercase.**
```r
        do.call(exa.readData, execArgs)
                              ^~~~~~~~
```
[R/EXADBI.R:6:2:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L6) *style:* **Commented code should be removed.**
```r
## First version written in 2015 by Marcel Boldt <marcel.boldt@exasol.com>
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:7:2:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L7) *style:* **Commented code should be removed.**
```r
## as part of the EXASOL R interface & SDK package. It may be used, changed and distributed freely with no further restrictions than
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:57:15:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L57) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAObject",
              ^
```
[R/EXADBI.R:58:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L58) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:82:15:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L82) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXADriver",
              ^
```
[R/EXADBI.R:83:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L83) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:97:44:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L97) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:146:15:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L146) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAConnection",
              ^
```
[R/EXADBI.R:147:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L147) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:148:10:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L148) *style:* **Variable and function names should be all lowercase.**
```r
    if (!dbIsValid(dbObj)) {
         ^~~~~~~~~
```
[R/EXADBI.R:148:20:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L148) *style:* **Variable and function names should be all lowercase.**
```r
    if (!dbIsValid(dbObj)) {
                   ^~~~~
```
[R/EXADBI.R:152:26:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L152) *style:* **Variable and function names should be all lowercase.**
```r
      db.version = paste(dbObj@db_prod_name, dbObj@db_version),
                         ^~~~~
```
[R/EXADBI.R:152:46:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L152) *style:* **Variable and function names should be all lowercase.**
```r
      db.version = paste(dbObj@db_prod_name, dbObj@db_version),
                                             ^~~~~
```
[R/EXADBI.R:153:16:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L153) *style:* **Variable and function names should be all lowercase.**
```r
      dbname = dbObj@db_name,
               ^~~~~
```
[R/EXADBI.R:154:18:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L154) *style:* **Variable and function names should be all lowercase.**
```r
      username = dbObj@db_user,
                 ^~~~~
```
[R/EXADBI.R:155:14:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L155) *style:* **Variable and function names should be all lowercase.**
```r
      host = dbObj@db_host,
             ^~~~~
```
[R/EXADBI.R:156:14:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L156) *style:* **Variable and function names should be all lowercase.**
```r
      port = dbObj@db_port
             ^~~~~
```
[R/EXADBI.R:165:44:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L165) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:189:47:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L189) *style:* **Put spaces around all infix operators.**
```r
                         fields = c(connection="EXAConnection",
                                             ~^~
```
[R/EXADBI.R:190:42:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L190) *style:* **Put spaces around all infix operators.**
```r
                                statement="character",
                                        ~^~
```
[R/EXADBI.R:191:45:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L191) *style:* **Put spaces around all infix operators.**
```r
                                rows_fetched="numeric",
                                           ~^~
```
[R/EXADBI.R:192:46:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L192) *style:* **Put spaces around all infix operators.**
```r
                                rows_affected="numeric",
                                            ~^~
```
[R/EXADBI.R:193:44:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L193) *style:* **Put spaces around all infix operators.**
```r
                                is_complete="logical",
                                          ~^~
```
[R/EXADBI.R:194:44:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L194) *style:* **Put spaces around all infix operators.**
```r
                                with_output="logical",
                                          ~^~
```
[R/EXADBI.R:195:40:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L195) *style:* **Put spaces around all infix operators.**
```r
                                profile="data.frame",
                                      ~^~
```
[R/EXADBI.R:196:40:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L196) *style:* **Put spaces around all infix operators.**
```r
                                columns="data.frame",
                                      ~^~
```
[R/EXADBI.R:197:48:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L197) *style:* **Put spaces around all infix operators.**
```r
                                temp_result_tbl="character",
                                              ~^~
```
[R/EXADBI.R:198:48:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L198) *style:* **Put spaces around all infix operators.**
```r
                                query_sent_time="POSIXct",
                                              ~^~
```
[R/EXADBI.R:199:39:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L199) *style:* **Put spaces around all infix operators.**
```r
                                errors="character",
                                     ~^~
```
[R/EXADBI.R:200:50:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L200) *style:* **Put spaces around all infix operators.**
```r
                                default_fetch_rec="numeric"
                                                ~^~
```
[R/EXADBI.R:210:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L210) *style:* **lines should not be more than 80 characters.**
```r
            "Add an int (the length of a newly fetched result set) to rows_fetched."
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:214:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L214) *style:* **lines should not be more than 80 characters.**
```r
#         close = function(commit=TRUE) {                              # dbClearResult is in row 880
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:215:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L215) *style:* **lines should not be more than 80 characters.**
```r
#           "Frees up all resources, in particular drops the temporary table in the DB."
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:218:17:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L218) *style:* **Commented code should be removed.**
```r
#               warning("Connection seems exipired.\n    ...it's gone...")
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:219:17:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L219) *style:* **Commented code should be removed.**
```r
#               return(FALSE)
                ^~~~~~~~~~~~~
```
[R/EXADBI.R:223:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L223) *style:* **Commented code should be removed.**
```r
#                 if(commit) message("Changes commited.")
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:224:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L224) *style:* **lines should not be more than 80 characters.**
```r
#             } else stop("Commit failed. Changes NOT commited. Closing aborted.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:225:13:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L225) *style:* **Commented code should be removed.**
```r
#           message("Closing connection...")
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:226:15:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L226) *style:* **Commented code should be removed.**
```r
#             res <- try(odbcClose(connection),silent=TRUE)
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:228:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L228) *style:* **Commented code should be removed.**
```r
#                 message("Connection closed.")
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:229:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L229) *style:* **Commented code should be removed.**
```r
#                 return(TRUE)
                  ^~~~~~~~~~~~
```
[R/EXADBI.R:231:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L231) *style:* **Commented code should be removed.**
```r
#                 warning("Closing not successful.")
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:232:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L232) *style:* **Commented code should be removed.**
```r
#                 return(FALSE)
                  ^~~~~~~~~~~~~
```
[R/EXADBI.R:234:23:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L234) *style:* **Commented code should be removed.**
```r
#                     warning(res)
                      ^~~~~~~~~~~~
```
[R/EXADBI.R:235:23:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L235) *style:* **Commented code should be removed.**
```r
#                     return(FALSE)
                      ^~~~~~~~~~~~~
```
[R/EXADBI.R:240:14:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L240) *style:* **Commented code should be removed.**
```r
            #close()
             ^~~~~~~
```
[R/EXADBI.R:247:15:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L247) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAResult",
              ^
```
[R/EXADBI.R:248:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L248) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:250:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L250) *style:* **Variable and function names should be all lowercase.**
```r
      statement = dbObj$statement,
                  ^~~~~
```
[R/EXADBI.R:251:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L251) *style:* **Variable and function names should be all lowercase.**
```r
      row.count = dbObj$rows_fetched,
                  ^~~~~
```
[R/EXADBI.R:252:23:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L252) *style:* **Variable and function names should be all lowercase.**
```r
      rows.affected = dbObj$rows_affected,
                      ^~~~~
```
[R/EXADBI.R:253:23:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L253) *style:* **Variable and function names should be all lowercase.**
```r
      has.completed = dbObj$is_complete,
                      ^~~~~
```
[R/EXADBI.R:254:19:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L254) *style:* **Variable and function names should be all lowercase.**
```r
      is.select = dbObj$with_output
                  ^~~~~
```
[R/EXADBI.R:262:44:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L262) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:272:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L272) *warning:* **no visible global function definition for ‘new’, Did you mean 'news'?**
```r
  new("EXADriver")
  ^~~
```
[R/EXADBI.R:277:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L277) *warning:* **no visible global function definition for ‘new’, Did you mean 'news'?**
```r
  new("EXADriver")
  ^~~
```
[R/EXADBI.R:287:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L287) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:298:47:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L298) *style:* **Variable and function names should be all lowercase.**
```r
setMethod("dbDataType", "EXAObject", function(dbObj, obj, ...) {
                                              ^~~~~
```
[R/EXADBI.R:299:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L299) *style:* **Variable and function names should be all lowercase.**
```r
  EXADataType(obj)
  ^~~~~~~~~~~
```
[R/EXADBI.R:303:13:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L303) *style:* **Variable and function names should be all lowercase.**
```r
  vapply(x, EXADataType, FUN.VALUE = character(1), USE.NAMES = FALSE)
            ^~~~~~~~~~~
```
[R/EXADBI.R:316:13:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L316) *style:* **Variable and function names should be all lowercase.**
```r
  vapply(x, EXADataType, FUN.VALUE = character(1), USE.NAMES = FALSE)
            ^~~~~~~~~~~
```
[R/EXADBI.R:323:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L323) *style:* **Variable and function names should be all lowercase.**
```r
    dbGetInfo(drv, "connectionIds")[[1]]
    ^~~~~~~~~
```
[R/EXADBI.R:385:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L385) *style:* **lines should not be more than 80 characters.**
```r
                        exalogfile = tempfile(pattern = "EXAODBC_", fileext = ".log"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:390:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L390) *style:* **lines should not be more than 80 characters.**
```r
                        connectionlcctype = Sys.getlocale(category = "LC_CTYPE"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:391:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L391) *style:* **lines should not be more than 80 characters.**
```r
                        connectionlcnumeric = Sys.getlocale(category = "LC_NUMERIC"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:395:3:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L395) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  {
  ^
```
[R/EXADBI.R:396:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L396) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(
    ^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:425:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L425) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(drv = dbDriver(drv), ...),
    ^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:425:28:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L425) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(drv = dbDriver(drv), ...),
                           ^~~~~~~~
```
[R/EXADBI.R:436:5:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L436) *style:* **Variable and function names should be all lowercase.**
```r
    EXACloneConnection(drv, ...),
    ^~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:448:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L448) *style:* **Variable and function names should be all lowercase.**
```r
dbCurrentSchema <- function(con) {
^~~~~~~~~~~~~~~
```
[R/EXADBI.R:449:10:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L449) *style:* **Variable and function names should be all lowercase.**
```r
  res <- sqlQuery(con, "select current_schema")
         ^~~~~~~~
```
[R/EXADBI.R:449:10:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L449) *warning:* **no visible global function definition for ‘sqlQuery’**
```r
  res <- sqlQuery(con, "select current_schema")
         ^~~~~~~~
```
[R/EXADBI.R:450:44:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L450) *style:* **Commas should always have a space after.**
```r
  con@current_schema <- as.character(res[1,1])
                                           ^
```
[R/EXADBI.R:457:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L457) *style:* **Variable and function names should be all lowercase.**
```r
EXANewConnection <- function( # change defaults also above
^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:481:11:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L481) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(dsn!="") {
          ^
```
[R/EXADBI.R:481:15:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L481) *style:* **Put spaces around all infix operators.**
```r
        if(dsn!="") {
             ~^~~
```
[R/EXADBI.R:482:38:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L482) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0("DSN=",dsn)
                                     ^
```
[R/EXADBI.R:484:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L484) *style:* **Put spaces around all infix operators.**
```r
        else if (exahost!="" & uid!="") {
                       ~^~~
```
[R/EXADBI.R:484:35:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L484) *style:* **Put spaces around all infix operators.**
```r
        else if (exahost!="" & uid!="") {
                                 ~^~~
```
[R/EXADBI.R:485:70:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L485) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0("DRIVER={EXASolution Driver};EXAHOST=",exahost)
                                                                     ^
```
[R/EXADBI.R:488:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L488) *style:* **lines should not be more than 80 characters.**
```r
            stop("Connect failed. Either DSN, host & db_user or a connection string must be given.\n
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:489:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L489) *style:* **lines should not be more than 80 characters.**
```r
                 Hint: No lazy declaration of onnection parameters - these have to be stated ' dsn=...'.\n
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:493:11:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L493) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(uid!="") {
          ^
```
[R/EXADBI.R:493:15:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L493) *style:* **Put spaces around all infix operators.**
```r
        if(uid!="") {
             ~^~~
```
[R/EXADBI.R:494:39:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L494) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                      ^
```
[R/EXADBI.R:494:47:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L494) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                              ^
```
[R/EXADBI.R:494:51:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L494) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                                  ^
```
[R/EXADBI.R:494:59:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L494) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                                          ^
```
[R/EXADBI.R:497:11:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L497) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(exaschema!="SYS") {
          ^
```
[R/EXADBI.R:497:21:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L497) *style:* **Put spaces around all infix operators.**
```r
        if(exaschema!="SYS") {
                   ~^~~
```
[R/EXADBI.R:498:39:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L498) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";EXASCHEMA=",exaschema)
                                      ^
```
[R/EXADBI.R:498:53:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L498) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";EXASCHEMA=",exaschema)
                                                    ^
```
[R/EXADBI.R:501:35:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L501) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";EXALOGFILE=",exalogfile)
                                  ^
```
[R/EXADBI.R:501:50:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L501) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";EXALOGFILE=",exalogfile)
                                                 ^
```
[R/EXADBI.R:504:35:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L504) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";LOGMODE=",logmode)
                                  ^
```
[R/EXADBI.R:504:47:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L504) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";LOGMODE=",logmode)
                                              ^
```
[R/EXADBI.R:507:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L507) *style:* **lines should not be more than 80 characters.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:507:35:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L507) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
                                  ^
```
[R/EXADBI.R:507:57:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L507) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
                                                        ^
```
[R/EXADBI.R:507:75:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L507) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
                                                                          ^
```
[R/EXADBI.R:507:99:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L507) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
                                                                                ^
```
[R/EXADBI.R:511:35:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L511) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";autocommit=",autocommit)
                                  ^
```
[R/EXADBI.R:511:50:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L511) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";autocommit=",autocommit)
                                                 ^
```
[R/EXADBI.R:515:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L515) *style:* **Put spaces around all infix operators.**
```r
        while (length(d)>0) {
                       ~^~
```
[R/EXADBI.R:516:39:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L516) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                      ^
```
[R/EXADBI.R:516:56:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L516) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                                       ^
```
[R/EXADBI.R:516:60:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L516) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                                           ^
```
[R/EXADBI.R:521:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L521) *style:* **Variable and function names should be all lowercase.**
```r
    con <- odbcDriverConnect(con_str)
           ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:521:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L521) *warning:* **no visible global function definition for ‘odbcDriverConnect’**
```r
    con <- odbcDriverConnect(con_str)
           ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:522:21:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L522) *style:* **Variable and function names should be all lowercase.**
```r
    exa_metadata <- odbcGetInfo(con)
                    ^~~~~~~~~~~
```
[R/EXADBI.R:522:21:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L522) *warning:* **no visible global function definition for ‘odbcGetInfo’**
```r
    exa_metadata <- odbcGetInfo(con)
                    ^~~~~~~~~~~
```
[R/EXADBI.R:524:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L524) *style:* **Commas should always have a space after.**
```r
    new("EXAConnection",init_connection_string = con_str,
                        ^
```
[R/EXADBI.R:526:52:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L526) *style:* **Commas should always have a space after.**
```r
        autocom_default = ifelse(autocommit == "Y",TRUE,FALSE),
                                                   ^
```
[R/EXADBI.R:526:57:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L526) *style:* **Commas should always have a space after.**
```r
        autocom_default = ifelse(autocommit == "Y",TRUE,FALSE),
                                                        ^
```
[R/EXADBI.R:527:56:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L527) *style:* **Commas should always have a space after.**
```r
        db_host = strsplit(exa_metadata["Server_Name"],":")[[1]][1],
                                                       ^
```
[R/EXADBI.R:528:67:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L528) *style:* **Commas should always have a space after.**
```r
        db_port = as.numeric(strsplit(exa_metadata["Server_Name"],":")[[1]][2]),
                                                                  ^
```
[R/EXADBI.R:530:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L530) *style:* **lines should not be more than 80 characters.**
```r
          regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:530:116:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L530) *style:* **Commas should always have a space after.**
```r
          regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE)
                                                                                ^
```
[R/EXADBI.R:530:120:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L530) *style:* **Put spaces around all infix operators.**
```r
          regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE)
                                                                                ~^~
```
[R/EXADBI.R:534:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L534) *style:* **lines should not be more than 80 characters.**
```r
              regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE))[[1]]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:534:120:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L534) *style:* **Commas should always have a space after.**
```r
              regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE))[[1]]
                                                                                ^
```
[R/EXADBI.R:534:124:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L534) *style:* **Put spaces around all infix operators.**
```r
              regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE))[[1]]
                                                                                ~^~
```
[R/EXADBI.R:551:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L551) *style:* **Variable and function names should be all lowercase.**
```r
EXACloneConnection <- function(drv, autocommit, ...) { # todo: parameters
^~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:551:54:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L551) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
EXACloneConnection <- function(drv, autocommit, ...) { # todo: parameters
                                                     ^
```
[R/EXADBI.R:553:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L553) *style:* **Variable and function names should be all lowercase.**
```r
    drv <- dbCurrentSchema(drv) # update schema metadata
           ^~~~~~~~~~~~~~~
```
[R/EXADBI.R:553:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L553) *warning:* **no visible global function definition for ‘dbCurrentSchema’**
```r
    drv <- dbCurrentSchema(drv) # update schema metadata
           ^~~~~~~~~~~~~~~
```
[R/EXADBI.R:556:25:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L556) *style:* **Commas should always have a space after.**
```r
    d <- data.frame(...,stringsAsFactors = FALSE)
                        ^
```
[R/EXADBI.R:570:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L570) *style:* **lines should not be more than 80 characters.**
```r
        con_str <- paste0(con_str, ";", s[[1]][1],"=", s[[1]][2]) # take the s parameter
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:570:51:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L570) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str, ";", s[[1]][1],"=", s[[1]][2]) # take the s parameter
                                                  ^
```
[R/EXADBI.R:586:31:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L586) *style:* **Commas should always have a space after.**
```r
    con_str <- substr(con_str,2,nchar(con_str)) # remove the initial semicolon
                              ^
```
[R/EXADBI.R:586:33:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L586) *style:* **Commas should always have a space after.**
```r
    con_str <- substr(con_str,2,nchar(con_str)) # remove the initial semicolon
                                ^
```
[R/EXADBI.R:588:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L588) *style:* **Variable and function names should be all lowercase.**
```r
    con <- odbcDriverConnect(con_str)
           ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:588:12:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L588) *warning:* **no visible global function definition for ‘odbcDriverConnect’**
```r
    con <- odbcDriverConnect(con_str)
           ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:590:1:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L590) *style:* **lines should not be more than 80 characters.**
```r
        stop(paste("EXACloneConnection error: failed to initialise connection.\nConnection String:", con_str))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:592:21:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L592) *style:* **Variable and function names should be all lowercase.**
```r
    exa_metadata <- odbcGetInfo(con)
                    ^~~~~~~~~~~
```
[R/EXADBI.R:592:21:](https://github.com/vsimko/r-exasol/blob/ad0394e820fcc8f49ea52375d8f55cb0abbb91e7/R/EXADBI.R#L592) *warning:* **no visible global function definition for ‘odbcGetInfo’**
```r
    exa_metadata <- odbcGetInfo(con)
                    ^~~~~~~~~~~
```


copy/paste detected


Interesting, doing this resolves that backtrace? From looking at the code it seems like there was some issue with serializing/deserializing the class, since there is no way for __dict to not be initialized in a normal class initialization.


I don't have anything against the abbreviation, however DRY out seems weird to me. At least I've never seen it before.


this._isObject(s) will always return `true` unless `s` is undefined. Meaning the if s is `null` the next line will throw an error.


outgrow (one word)


Hi,

I don't mind adding the dynamicAccess classloader to the Kryo serializer if this fixes the loading of OSGI provided classes. But why downgrade the akka dependencies to 2.3 ?

I think we should patch version 0.3 and release new patch version if we need this support working with Akka 2.3.

BTW, the difference of 0.4.0 and 0.3.4 was releasing it agains Akka-2.4 and only Scala-2.11 while 0.3 was released agains Akka-2.3 and Scala 2.10 + 2.11


You can rewrite these using model.setNull:
```javascript
model.setNull('min', 0);
```

Basically, setNull checks if the value is set or not, and if it is not set (and only then), it sets the passed value. See http://derbyjs.com/docs/derby-0.6/models/setters


Well you already process the data a bit, there are some properties that would be useful to test IMO


Oops, it's "revert Volume music control"


it's another responsibility


You could also use Literal.newLiteral("somename") as Input


Maybe. :smile: But you didn't describe a parallel (children / childItems) level-order traversal, did you? That's what I meant.


See: http://endlessparentheses.com/define-context-aware-keys-in-emacs.html


Note that this does not necessarily make thumbPosition reactive to changes on min (since min is not passed in as a dependency).


Great! Thank you! :)


args aren't used?


extract to `parseToken`


Not sure if this makes sense here, but would this be better:
List<StatsListener.OperatorResponse> list = o.stats.operatorResponses;
if (list == null ) {
    o.stats.operatorResponses = new ArrayList<>();
} else {
    list.clear();
}


Why lose the full stop?


Jeb


Here a RuntimeException has to be thrown. The IllegalArgumentException is thrown When try to add a rule with existing ruleUID. The different exception are used in Rule Resource to differentiate two case and return different response code for rest requests. (400 - missing parameter, 409 - dublicate rule uid)


Not sure why it should not throw


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1214.7a2c3ab9339cf1e6a91e7791ffba7ce26110944b](http://build.palaso.org/viewLog.html?buildId=67867&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 824, ignored: 7 Build time: 00:21:06


I'm totally fine with the model being loaded before everything else, but we should probably generalize it, and load all model classes (those without ".service." ".controller." ".module." ".config." ".run." infix). Then we could get rid of that nasty ```<%= temp_dir %>/ts/components/core/elementsConfig/*.js'```


anyway, you can write `assertThat(blabla).isEmpty()`


Aee


I always have the issue on Windows 7 pro using the doctrine/cache version v1.5.4. I was obliged to decrease the value to 254 to work fine.


should be NotAnAccesTokenException


Nice! :) Quite much cleaner.


This is an outdated commit...


a lot of tests are missing


instead of ```path?:string[]``` and than later setting it to an empty array, isn't it easier to just use a default value like so: ```convertPropertiesToControls(path:string[] = []): ControlToolboxElement[]```?


It's valid qt logic.
please use doc ->  http://doc.qt.io/qt-5/qtnetwork-fortuneclient-example.html


There should actually always be an empty line break at the end of each file, for cleaner commits.


Does this work?

Does installation data suite work?


Sorry about that. This is an outdated commit, the actual one is at https://github.com/briantopping/akka-kryo-serialization/commit/aac5e2677b08e22a78a921125b7de8648b3f3880.


Goodness knows how that happened...


This is a death rattle.  Github is based around forking... so calling people who fork and revive a project 'retards' shows me the battle is long over, and I'm moving to the new repo.


I think I see what you mean. while the term epoch seemed natural for RGWRealm, we're now making RGWPeriod track two different epochs that are unrelated and have no basis for comparison

i'm not sure what the best solution is here, but i think it would help to be more explicit when dealing with realm epochs. for example, renaming `Cursor::get_epoch()` to `get_realm_epoch()`, renaming local variables from `epoch` to `realm_epoch`, etc.


dot com*


##Below are two functions that are used to create a special
##object that stores a matrix and caches its inverse.
##The first function, makeCacheMatrix creates a special
##“matrix”, which is really a list containing a function to

##set the value of the matrix
##get the value of the matrix
##set the value of the inverse
##get the value of the inverse


/recovery/update.img ? :)


yeah thanks for noticing :) fixed


Try to stick to conventions (https://en.wikipedia.org/wiki/Digital_biquad_filter) - b0,b1,b2 are forward coefficients (FIR), a1,a2 are feedback coefs.


Can we call it something like `sync_mark` rather than `k`, to make it clear what the keys represent?


i'm going back on forth on whether it's better to sacrifice the nice default case for the confusing multi-edit case. fine leaving as-is for now and revisiting if people report confusion


Hmm, there is a nasty failure, but nobody knows about it. IMO there are 2 steps to avoid such problems:

1. we need to make sure queue is safe. I think only we should not send the exception, we should send only it's class (or info whether it's SKIP, FAIL, OTHER), traceback and string representation. We don't actually need the exception itself (we currently do this for old-style exceptions).
2. we should include your patch, but we need to notify the user. I think setting the test as `WARN` with information about unparsable message from the test would be appropriate.


makeCacheMatrix <- function(x = matrix()) {
        i <<- NULL
        set <- function(y) {
                x <<- y
                i <<- NULL
        }
        get <- function() x
        setinverse <- function(inverse) i <<- inverse
        getinverse <- function() i
        list(set=set,get=get,setinverse=setinverse,getinverse=getinverse)
}


This commit is broken, the attachment content has been placed out of the XML.


test


Just small note: Real custom elements should have hyphen somewhere in the name. Maybe consider to change the example below to something like `<cool-message>`.

https://www.smashingmagazine.com/2014/03/introduction-to-custom-elements/


Class objects/variables/fields can't be shared in a multi-threaded application unless the shared objects are either immutable or thread safe. Even if they are thread safe, it could lead to other kind of bugs in that changes to the clone leads to changes in the original and vice verca. Immutable objects are safe as they can't be changed, and strings are also safe because they are by Java definition immutable (every time you change a string a new one is created and the old discarded).

I looked a bit more at your change though, and I see that it's only semantical so it doesn't really affect anything. Further discussion on the issue should be done in #795.


Btw we also get data from the queue in `early_status`. How about creating `self.read_queue` which wraps the `self.queue.get` and takes care of the error handling?


salam azizam


You would, indeed, get a new instance for each JRuby.  But each instance would be a thin proxy wrapping around the shared `puppet_server_terminus_shared_context` object, which would be injected by Puppet Server...


Ook `is_plugin_active('mlang/mlang.php')` toevoegen


:tada: :confetti_ball:


It would be better to separate filter state from filter coefficients (probably by passing two pointers). But that can be implemented later.

Also with higher order filter, y1,y2 is identical to x1,x2 of next section. Some memory can be saved by using this property ...


Hi, @dreab8
I have created the small project that will show the issue and pushed changes on github.
https://github.com/EugenMaysyuk/hib
You can run Main class to see the issue.


PS: Did you wanted to use your gmail address?


I think so as well - garbage in garbage out.


Yes.


I'm fine with this, but if the goal is to have a shortcut, I wonder if wouldn't be better to simply make new commands for the vehicles

    waf arducopter
    waf ArduCopter
    # or whatever variant

would be shortcut for `waf build --target bin/ArduCopter`. Besides being smaller (a feature for shortcuts), avoid special case for the relation between name and target for our targets.


I verified that it's backwards, both at line 54 and 71.  If you switch them the email look up works normally.


Grunt tasks are queued in a task list and run in the order called. They need to be seen as such, and not just as named tasks that can be run from the command line. In this case, replacing the call to the 'npm-publish-report' task with a function call would result in the function call being made prior to the loop of "npm-publish" tasks completing. So without restructuring the logic of "npm-publish", I think this is an appropriate and expected use of a grunt task.


Closes #13


I pushed an update


اقا اینجا چت نکن لدفا
همه دارن ما رو می بینن


`daysArray.map(...)`?


Aside from those comments LGTM


@sevoku: Could you please move it somewhere so we get color changes on skin change? Currently it keeps the color which is set on start.


With gyro sync, samples are exactly spaced. Make sure that this does not introduce noise  (passed dT is constant)


Hi Paul, a few observations for this commit.
- /var/log path in not accessible on shared hosting . Defaulting there may cause issues with people trying to install there.
- Currently emocms has a log file size limit of about 250Mb (log is reseted reaches this value).
- $config_file_version is to signal structural changes on the config file variable names, not the default values.


LGTM


Hi @EugenMaysyuk ,

I have created [https://hibernate.atlassian.net/browse/HHH-10437]

Thanks


Do you think anyone was relying on the old class name? If so, this is backwards-incompatible. If not, no worries.


Hi @lepidus, this doesn't fix the issue for me. You'll need to modify this to DATE(date_posted) <= CURRENT_DATE.

Likewise for the expiration date, you'll need DATE(date_expire) > CURRENT_DATE.


It occurred to me that that patch might not be necessary after I posted my previous comment.

I don't think the patch does any harm, even though it's not necessary. The minimal requirement that upstream specified on the gtk2 version was unnecessarily restrictive, so the patch corrects an inaccuracy. Also, the patch would be useful if it makes sense for pre-hipster versions of OI.

On the other hand, the fewer patches, the better…


Lors de la sauvegarde, on n'a pas accès aux Maps ni aux Events, donc les interrupteurs locaux doivent être rangés dans Partie.
Les interrupteurs ordinaires au contraire sont déjà dans Partie, donc on peut utiliser un tableau de booléens pour l'implémentation.
Peut-être qu'un arbre permettrait d'augmenter les performances de l'accès aux interrupteurs locaux...


That's fine. I was just cleaning up the mod info to show how you can hard code it without mcmod.info file needed.

On Jan 11, 2016 10:38 PM, iLexiconn <notifications@github.com> wrote:

"jcanimationapi" is not a required mod anymore, and I'm going to remove it in my next commit.

—
Reply to this email directly or view it on GitHub<https://github.com/TimelessModdingTeam/JurassiCraft2/commit/a0b67c77aa4fd85d79b64b16025ba1fb584387a4#commitcomment-15388882>.


If you're doing this because you're having issues with the `/unregister` endpoint, Pushwoosh put it on a `POST` http method, not a `DELETE`.

It only took weeks of slow back and forth with their super slow support to finally figure this out :(

I came across this and just thought I'd mention it.


Changed the update interval to 90s to avoid constant flickering.