Место у нас задается при инициализации и занимается методом take_space. Зачем нарушать инкапсуляцию и открывать переменные для изменения внешнему коду? тут должены быть ридеры


IIRC you had some issues with callbacks, and I think we agreed that you would use a task in order to defer execution to a different context, but I don't see anything related to that here, did I miss something?


а если место закончится?


Dear Mr. icarpis:
I used the code you changed with "reset",it work well.
But the fps is lower than before, I set fps=30,but with this pkg, the raw data(/camera/depth_registered/points)fps is down to 10fps, which cannot applied in real machine, do you have any improvement?
Hope your reply!
Qiuchen


Hi,

Continuing on these Thread, I want to use the save as you proposed:

save saved_simulation_file('saveSimu.gsim', [simulation]);

Basically I tried to implement a constructor:

@doc("Constructor for saved simulation files from a list of agents: it is
used with aim of saving a simulation agent.")

public GamaSavedSimulationFile(final IScope scope, final String pathName,
final GamaList<IAgent> contents) {

* super(scope, pathName, null);*

IAgent agent = contents.firstValue(scope);

// Set first the metadata

setMetadata(scope);

// Set the buffer

final String serializedAgent = ReverseOperators.serializeAgent(scope, agent
);

IList<String> c = GamaListFactory.create();

c.add(serializedAgent);

setContents(c);

// fillBuffer(scope);

}

My point is that I was not able to find a constructor of GamaFile able to
create a file that does not exist.
So everytime I try I get an exception as the file does not exist.

Did I miss something ?

Cheers

Benoit

Le mar. 14 août 2018 à 14:08, Alexis Drogoul <notifications@github.com> a
écrit :

> Sorry, I mixed fillBuffer and flushBuffer ! fillBuffer is used for
> reading, the other for writing (as the buffer is filled with what's passed
> as the second argument).
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/gama-platform/gama/commit/ec6cf846c58fd4c0e073528dac634b44635126fb#commitcomment-30070999>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABe7Ssy71HW8b0H1un2MTXZ_MxSppD7pks5uQr3NgaJpZM4V6szY>
> .
>


Тут этот метод вообще не нужен. Он такой-же как метод в родителе


yes, thanks


Зачем тут ридеры? Они ведь одинаковые в обоих подклассах.


I think it would be clearer if you split xen_load_helper into smaller functions that can be called from the different contexts. I think there's an unnecessary amount of churn, like calling a load_helper with a NOLOAD type. Quite a lot of this can be avoided by splitting the load helper into smaller pieces that can be called independently.


А зачем записывать тип отдельно? Просто сразу нужно отправить "захардкоденый" тип в родительский класс


async/await is a bit easier to use than promises


Тут тоже ридеры не нужны


이 분 조만간 nodeJS 씹어드실분 ㄸㄸ


I think you can use xentag->domid directly here and get rid of the local domid variable. In general if a variable is used in a single place don't make it local.


И тип не нужно пытаться сохранить в подклассе, а переотправить сразу родителю


merging all from git


Fair enough. I find the unrolled version better readable, if we follow a near-declarative style. A similar thing would be in ther ti99_8 driver. I already simplified the GROM instantiation, however.


Both refs and domid are used in a single place, so there's no need for this local variables.


The URI link you have provided differs from the URI link silverpeas is requesting.

Your link.
**https://www.silverpeas.org/nexus/content/groups/silverpeas/_org/silverpeas/jcr/jackrabbit-jca/2.14.4/_**

Silverpeas requesting link:
**http://www.silverpeas.org/nexus/content/groups/silverpeas/_org/apache/jackrabbit/jackrabbit-jca/2.14.4/_**

It is picking up the link **http://www.silverpeas.org/nexus/content/groups/silverpeas** from "settings.xml" file in root/.m2 folder(inside the container) but after that it is appending the rest of the link.

I am not able to get from where the the rest of the link is being appended.
I have already checked the pom files and did some changes but couldn't succeed .
where should I do the changes locally to force silverpeas to pick the right link for downloading the rar file.


本件は、リポジトリの移動および、下記の修正を行いました。
- #1
- #2
- #3
- #4
- #9


i need to learn this travis.yml file please have class on that for me @8317259388


Start projekt


Ugh. Maybe we can revert?


reverse() ,out of bounds


Looking good! Thanks for pushing :)


Oh yes true, I didn't realize


I don't know... the impact doesn't seem that large, and I consider absolute paths OK in those variables.


Sorry, I have no idea what that comment means. If you are part of a class at some college or university, please contact them.


could just say
```python
else:
```
here


Changes made here seem good according to the Code Style Guidelines.
last two lines look like skipped as these need to change from "ui" to "payFineUi".
Reviewed by Gurpreet Gill


已经改用 rowKey https://github.com/react-component/table/commit/6f4d0596a3e80e328d895815bf1994aa5d477759


Changes made here seem good according to the Code Style Guidelines.
Reviewed by Gurpreet Gill


Ah, just a mistake. Take it easy then :)
I understand keeping/maintaing multiple parallel branches is a bit of a nightmare.
No worries!


@z3r0v01t Do you need to decipher some text?
And ssspbank.so can not be anything to open. I do not know English, I'm sorry for the mistakes.


@fabiencastan `-DEigen3_DIR:PATH=${BUILD_DIR}/eigen_build/share/eigen3/cmake` is repeated twice, is it on purpose?


We might consider using absolute paths only when cross-compiling, or something like that.  (I guess that's the case where confusion happens most easily and it's better to use absolute paths then.)


Programmation évènementielle : Le serveur est à l'écoute des évènements. Ainsi, à chaque évènement, ici une requête GET, le serveur adopte le comportement défini, c'est à dire exécute la fonction associée à l'évènement.


@u481915g コミットメッセージを英語で書く姿勢は、良いですね！日本語でも良いとする会社も多いです。


Could you throw a ValueError here?


This should use logger.info()


@u481915g 一行がちょっと長いので、折り返して記述すると良さそうです。


Why was the `FNV offset basis` for 32 bits from https://en.wikipedia.org/wiki/Fowler%E2%80%93Noll%E2%80%93Vo_hash_function#FNV_hash_parameters, `2166136261 = 0x811c9dc5`, not used instead?


Hot


@u481915g こちらも一行がちょっと長いので、折り返して記述すると良さそうです。
また、ロケットハッシュ記法は、Ruby1.8時代の古い記述方式のため、シンボル表記に書き換えましょう。

```
recipes: :user
```


No u


no


Maybe just call this DynamicSamplingLossCalculator? Having something called wrapper in the config file is a bit confusing.


@u481915g ここもですね。
{ } の前後には半角スペースを入れるのが良さそうです。


Thanks for this!


@u481915g シンボル表記にできそうです。


go for basic understanding first.


could you add some comments about what exactly is happening below?


@u481915g HTMLのインデントは、半角スペース2つに統一してください。
https://google.github.io/styleguide/htmlcssguide.html#Indentation


go form basics.


Thanx.


wow who is dis professional


Fixes #3638


@u481915g ロケットハッシュ記法が散見されます。直しておきましょう。


@u481915g カンマの後の半角スペース必要です。


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/tag-group-permission-keys-are-being-translated-and-shouldnt/90507/5


Moved to #169.


@u481915g 現場に配属されてから、他人が読みやすいコードにするために、スタイルガイドは遵守しましょう。


@u481915g コメントは削除するなど、何かしらのルールが必要です。


@u481915g []の前後のスペースなどについては、こちらをチェックしておくと良いです。
https://github.com/fortissimo1997/ruby-style-guide/blob/japanese/README.ja.md#spaces-braces


@u481915g ここに加えていますが、どうやら登録されているような気がします。


It would be nice to add some short module-level documentation about what this dynamic sampling is doing on a very basic level (selecting training data based on the previous training losses).


@FLavalliere To the best of my knowledge the haproxy maintainers do not actively monitor GitHub comments. I thus sent a mail with a link to your comment to the official mailing list: https://www.mail-archive.com/haproxy@formilux.org/msg30990.html

You might consider subscribing in case there are follow up questions.


@u481915g コミットメッセージの内容と作業内容が不整合な状態のため、気をつけましょう。もっとコミット単位を細かくした方が、レビュー時などにわかりやすいです。


gedaan want goede reden!!!!


@u481915g 不要なアセットファイルは削除しましょう。


'texte' is french and 'text' is english


@u481915g このライブラリは Ruby 1.9.2 で net/http にマージされたようです。
https://docs.ruby-lang.org/ja/latest/library/net=2fhttps.html


Qui Suis-je?
Agé de 47 ans , je suis ingénieur d'étude et directeur adjoint d'une unité expérimentale de l'INRA, l[a Plateforme d'infectiologie expérimentale](https://www6.val-de-loire.inra.fr/pfie)


no space before colon in english


missing C in uppercase


fix-eio-wind-angles (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (3021 of 3021 tests passed, 1 test warnings)

Messages:

 * 1 test had: EIO diffs.

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


https://github.com/SupremeMortal/EssentialsNK/commit/13803edecfc6d096a81e37ecee9c309e528e6f02#diff-8406b93b5e0527b6f6361a1f21eac57dR27 Should be || here
https://github.com/SupremeMortal/EssentialsNK/commit/13803edecfc6d096a81e37ecee9c309e528e6f02#diff-8406b93b5e0527b6f6361a1f21eac57dR47 player.sendMessage?
Also there should be a message when succeed.


@u481915g 自分が投稿したレシピでなくても削除できてしまって良いですか？


@u481915g 使わないヘルパーファイルは削除しておきましょう。


the 1.0.0 changelog includes

```
It is now recommended that Flatpak packages add a dependency on
p11-kit-server, as this allows apps to access host
certificates. However, this does not need to be a hard dependency.
```


@u481915g インデントにご注意を。公式ドキュメントもみておいて下さい。
https://api.rubyonrails.org/v5.1/classes/ActionView/Helpers/FormHelper.html#method-i-form_with


The same for ignore command.


@azat I'sorry, i forgot to reply , thanks for your help , my english is bad, sorry!


![](https://github.trello.services/images/mini-trello-icon.png) [linreg_ridge_lu](https://trello.com/c/XTfgugMy/161-linregridgelu)


@u481915g ロケットハッシュ記法が気になりました。


accepted.


Thank you - they all look ok


@Henry2o1o
Das gelöschte "privacy_apps_ (enable/disable)_ header_title" ist eine doppelte Zeile und existiert bereits.
(The deleted "privacy_apps_ (enable/disable)_ header_title" is a duplicate line and already exists.)

https://github.com/berlinux2016/MIUI10/commit/d7c9ea775d41d26baa167742af2f5cc2c80f697a#diff-80c525a0ddce2710e6abbd0214f8ba08L285
https://github.com/berlinux2016/MIUI10/commit/d7c9ea775d41d26baa167742af2f5cc2c80f697a#diff-80c525a0ddce2710e6abbd0214f8ba08L289


fix-eio-wind-angles (jasondegraw) - 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.png)


@froschdesign I know but why remove the option for console? two different options make me confuse.


You're right. It's has to be defined.


浅田
細かいですが、if文を並べるより、if～elseifとしたほうがごくわずかですが、処理が早いです。
また、可読性の観点から見てもそのほうが良いと思います。


这个performance差别大概有多少，感觉没啥变化


could you add the `break;` within {} allmann-style?


Hi! This is the friendly automated conda-forge-webservice.

I updated the Github team because of this commit.

- @tbeu was added to conda-forge. Welcome to conda-forge!
  Go to https://github.com/orgs/conda-forge/invitation see your invitation.

You should get push access to this feedstock and CI services.

Feel free to join the community [chat room](https://gitter.im/conda-forge/conda-forge.github.io).

NOTE: Please make sure to not push to the repository directly.
      Use branches in your fork for any changes and send a PR.
      More details [here](https://conda-forge.org/docs/conda-forge_gotchas.html#using-a-fork-vs-a-branch-when-updating-a-recipe)


k


Good to see you teaching someone else to code Jimbo but make sure your code is consistent (better to get them into good habits early) either put brackets on the same line or on the next line but don't mix and match, see Cat setOwner & makeNoise.

One moment you're putting curly brackets on different lines in your php and the next thing you know you'll be dropping the curly brackets entirely from your _if_ statements in your Javascript! who know's what problems that will cause ;)

Hope you're doing well Jimbo!
Eugene


The filter() is not filtering results. It lets even those with indexOf != -1 through. On my way in to fix it 🏎


試しにコメントしてみる


@u481915g コミットメッセージは、開発メンバーとのコミュニケーションに使われます。しっかり考えて記述しましょう。


Since this release, Jackett no longer runs:
https://drone.spritsail.io/spritsail/jackett/20/4

Is the build broken or are we missing some mono deps?


aaa


Let me know if you still need help :)
I'll take a look later if necessary ;)


@u481915g シンボル表記の方がシンプルに記述できます。

```
      render :new
```


Thanks Filip! I think this change should work for us. I had forgotten about the ObjectProvider wrapping classes.

I found a couple of typos in a the following readme sentence.
It is not possible to defxine the custom IdGenerator ...

I believe it should be the following instead.
"It is **now** possible to **define** the custom IdGenerator ..."


"Integrated squared 2nd derivative of the true function for each group level versus that for the functions estimated by model 4 (red) and model 5 (blue), indicating substantial undersmoothing for low-variability curves by model "

Requires further explanation, I think it's not super-clear why that's a useful measure from the caption alone (I'm from the school of "you should be able to understand everything from the caption", which I realise everyone else might not be :))


abc


This could be merged in after the `bam` intro.


Muy buen cambio, me gusto como refactoraste este codigo.


@probablyup I am working on a different PR onto develop. Little confused on why some of the exports were removed.


commented this out for now, absorb into "computational appendix" thingo?


commented this out for now, absorb into discussion, briefly?


Debemos cambiar igual el po/json en los lenguajes para que tambien afecte al menu del footer.


pick up in discussion


Two additional spaces


Important points:
- Pages are also Elmish components. Basic structure is: Types (model and message), State (init and update functions) and View function.
- The component must be integrated in the hierarchy. In this case, the App component needs references to Users' model, message, update and view.
- As this is Page, we integrate it with the hash-based SPA navigation. Add it to the Routes type and then to toHash and parser functions. Finally we deal with the new page in App's urlUpdate function.


@guaka this seems a bit out of place here without any context?

Note that there's a section in [INSTALL.md](https://github.com/Trustroots/trustroots/blob/2c54afc8501877ee1491b8ef052ecada1293e587/INSTALL.md#running-tests) for how to run tests.


Vulnerable


I'm ok with that


Thank you, i figured out how to subscribe to the mailling list and also be able to post issues there :) I had not found out how to subscribe before... :)


`badDescription` 1st argument is a part of an error message, so it should always start with either an upper case or a lower case letter. Make sure all such messages follow uniform syntax.


Added the final advertisement code


Sorry. I'm not sure how that got in here or what it's for even. I didn't consciously add it. As far as I'm concerned it can be removed.

Should I make another commit to remove it?


浅田
途中でのリターンはエラーの原因となりうるので原則禁止です。


AdminBrokerProcessor#getAllDelayOffset中可能有同样的隐患，代码如下：

```java
    private RemotingCommand getAllDelayOffset(ChannelHandlerContext ctx, RemotingCommand request) {
        final RemotingCommand response = RemotingCommand.createResponseCommand(null);

        String content = ((DefaultMessageStore) this.brokerController.getMessageStore()).getScheduleMessageService().encode();

```


generateQuestionComponent: function (pluginId) {


Hmm when I look at dropwizard-bom 1.3.5 it lists Jersey version as 2.25.1:
https://repo.maven.apache.org/maven2/io/dropwizard/dropwizard-bom/1.3.5/dropwizard-bom-1.3.5.pom
> <jersey.version>2.25.1</jersey.version>

And when I run a dependency list on dropwizard-jersey everything comes back as version 2.25.1:
```
➜  ~ mvn dependency:list -f ~/.m2/repository/io/dropwizard/dropwizard-jersey/1.3.5/dropwizard-jersey-1.3.5.pom | grep org.glassfish.jersey
[INFO]    org.glassfish.jersey.bundles.repackaged:jersey-guava:jar:2.25.1:compile
[INFO]    org.glassfish.jersey.containers:jersey-container-grizzly2-servlet:jar:2.25.1:test
[INFO]    org.glassfish.jersey.core:jersey-client:jar:2.25.1:compile
[INFO]    org.glassfish.jersey.containers:jersey-container-servlet:jar:2.25.1:compile
[INFO]    org.glassfish.jersey.containers:jersey-container-grizzly2-http:jar:2.25.1:test
[INFO]    org.glassfish.jersey.ext:jersey-metainf-services:jar:2.25.1:compile
[INFO]    org.glassfish.jersey.core:jersey-server:jar:2.25.1:compile
[INFO]    org.glassfish.jersey.core:jersey-common:jar:2.25.1:compile
[INFO]    org.glassfish.jersey.test-framework:jersey-test-framework-core:jar:2.25.1:test
[INFO]    org.glassfish.jersey.media:jersey-media-jaxb:jar:2.25.1:compile
[INFO]    org.glassfish.jersey.ext:jersey-bean-validation:jar:2.25.1:compile
[INFO]    org.glassfish.jersey.test-framework.providers:jersey-test-framework-provider-grizzly2:jar:2.25.1:test
[INFO]    org.glassfish.jersey.containers:jersey-container-servlet-core:jar:2.25.1:compile
```


Please update
![image](https://user-images.githubusercontent.com/16340573/44399125-a3d8fa80-a546-11e8-98d0-1d3d44fb0229.png)


Can top-level functions in scripts have contracts?


onclick="EkstepRendererAPI.dispatchEvent(question.pluginId + ':show', {'event':event,'src': \'<%= question.data.question.image %>\'})


This would be `/user/johnmayer/posts/new`


There is another thing with this default team. What if that team is archived? I don't this it makes sense to default to inactive team (I mean `'sales_team.team_sales_department'`). Because when you are accessing object by its external ID, it does not check whether that object is active or not.

And why not return `crm.team` object all the time? It would be more consistent then returning `False`. I mean instead of returning `False`, it would be better to return `self.env['crm.team']`.


Note, this issue would have been caught by `golint`. Is the linter currently configured to run on Travis-CI?

```go
// Package bar provides ...
package foo
```

```
u@x61s ~/D/g/s/p/foo> golint .
foo.go:1:1: package comment should be of the form "Package foo ..."
```


problems can occurs with zone.js


PAIA auth **login** changed to POST


onclick="EkstepRendererAPI.dispatchEvent(questionUnitPluginId+ ':show', {src:"<%= question.data.question.audio %>"})"


以後参考にします！今回はシードファイルは結局使わないことになったので削除します。


Why? Do you plan to support Windows?


```diff
-until this repository unless this repository is using semantic versioning
+until this repository is using semantic versioning
```


worng


I made a PR: https://github.com/rbenv/ruby-build/pull/1226


@anagrius Didn't we have a nice way of linking to query functions? One day we should build a nice hover pop up-thingie which shows a brief about the function, like they have on wikipedia
![image](https://user-images.githubusercontent.com/81906/44399435-ddf6cc00-a547-11e8-9bb2-7961a5ee541f.png)


```diff
-If a zc module A depends on another zc module B in terms of imports, then
+If a zc module `A` depends on another zc module `B` in terms of imports, then
```


I just realize that you will have to update the icon cache manually or to reboot your computer for now:
`sudo gtk-update-icon-cache -f /usr/share/icons/hicolor/`

Did i said that i'm blind without being able to test it myself? ;)


@kaileu Ahh, thanks for the clarification. I did not realize that the HLS streams were of a lower quality than the chromecast ones. Sigh.

I think this function should return all available stream types; then the client (Kodi in our case) will have to figure out what the most sane defaults/settings are for users.


read me


@JoshVStaden  This is how I set up my paths.

```bash
# CUDA 8
# export CUDA_PATH=/usr/local/cuda-8.0
# export PATH=$CUDA_PATH/bin:$PATH
# export LD_LIBRARY_PATH=$CUDA_PATH/lib64:$LD_LIBRARY_PATH
# export LD_LIBRARY_PATH=$CUDA_PATH/extras/CUPTI/lib64/:$LD_LIBRARY_PATH

# CUDA 9
export CUDA_PATH=/usr/local/cuda-9.0
export PATH=$CUDA_PATH/bin:$PATH
export LD_LIBRARY_PATH=$CUDA_PATH/lib64:$LD_LIBRARY_PATH
export LD_LIBRARY_PATH=$CUDA_PATH/extras/CUPTI/lib64/:$LD_LIBRARY_PATH

# CUDNN 6.0 (CUDA 8.0)
# export CUDNN_HOME=/usr/local/cudnn-6.0-cuda-8.0
# export C_INCLUDE_PATH=$CUDNN_HOME/include:$C_INCLUDE_PATH
# export CPLUS_INCLUDE_PATH=$CUDNN_HOME/include:$CPLUS_INCLUDE_PATH
# export LD_LIBRARY_PATH=$CUDNN_HOME/lib64:$LD_LIBRARY_PATH

# CUDNN 5.1 (CUDA 8.0)
#export CUDNN_HOME=/usr/local/cudnn-5.1-cuda-8.0
#export C_INCLUDE_PATH=$CUDNN_HOME/include:$C_INCLUDE_PATH
#export CPLUS_INCLUDE_PATH=$CUDNN_HOME/include:$CPLUS_INCLUDE_PATH
#export LD_LIBRARY_PATH=$CUDNN_HOME/lib64:$LD_LIBRARY_PATH

# Latest NVIDIA drivers
export LD_LIBRARY_PATH=/usr/lib/nvidia-396:$LD_LIBRARY_PATH
```


@eladsal we will discuss as a community and come to a consensus regarding the contribution workflow.


This not work in asp.net 2.1!! We can remove the ~?


I can see how this applies to 6 or 7 digit numbers, but how does it work for 9 or 10 digit numbers?


id -> ID


Zmienne w klasie powinny być prywatne


@marshyonline @davidmckenzie Do you guys mind taking a peek at this before i submit a PR.

It seems to work fine for me (except for the reloading the web app) any thoughts/improvements you can think of?

Cheers


Zmienne w klasie powinny być prywante


@StephanOsthold
Um doppeltes return und Redundanz zu vermeiden:
```
my $result = $this->{states}->{$state}->{$row};
$result = unescapeEntities($result) unless $noEntityEscape;
return $result;
```


id -> ID

How about logging the new ID instead of saying 'something unique'


I guess Docker folks won't check INSTALL.md?

On Tue, 21 Aug 2018 at 13:29, Mikael Korpela <notifications@github.com>
wrote:

> @guaka <https://github.com/guaka> this seems a bit out of place here
> without any context?
>
> Note that there's a section in INSTALL.md
> <https://github.com/Trustroots/trustroots/blob/2c54afc8501877ee1491b8ef052ecada1293e587/INSTALL.md#running-tests>
> for how to run tests.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/Trustroots/trustroots/commit/5562b83dbab98a18bfb7e3a25d0145864aaf326f#commitcomment-30234573>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AADVyPH8F9tyt8UORn9wdcZqVpuXYo2Cks5uS-8BgaJpZM4WFl6P>
> .
>
--
sent from my phone


fix-eio-wind-angles (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1721 of 1721 tests passed, 0 test warnings)

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


var a = 111
var foo = function () {
  var a = 222
  a
}
a
foo()
/*outputs:
111
222
*/


>So .. what do I have to do on fedora now? Pass -Dubuntu-bionic-patched-vte=false to meson?

Yes.

>(Also, usually the "normal" case would be the default, but that's another issue.)

The choice was between breaking it for all downstreams by reverting the commit, breaking it for devs on elementary OS by only fixing the build in packaging with a patch there, or fixing it for elementary OS devs while giving downstreams a convenient build flag (this solution). I don't see any better alternative.


Could we also release a new version of the gem? Let me know if you need anything from me to be able to do so. I guess by semver we should bump the major version?


Compreensão*


What about anonymous function literals?


numbers -> digits


sad


This should be fixed by https://github.com/openshift/openshift-ansible/pull/9695, please verify this resolves issues in your case


@oznetmaster That would make it cease to work with `ImmutableArray`. `actual` might not be a reference type, so comparing with `null` is legitimate.


Agreed. What was working when we were 2-3 committers might not be suitable now and we have to change they way we do things.


т.к. stock и price ссылочного типа, было бы неплохо проверять их на null во избежание NullPointerException


@oznetmaster That would make it cease to work with `ImmutableArray`. `actual` might be a reference type, so comparing with `null` is legitimate.


At this point in time we wanted to concentrate on features, not platform support. The instructions removed here are no longer applicable.

Anyways, Windows support is now an ongoing project with the new system. Sadly I don't have a Windows computer at my disposal and some issues still need to be figured out. I'll try to hurry up a little bit since there is demand.


@jannau — Think we should move this back where it was, or should we just alter `stm32f1.py`?


This seems to break API when cmd parameter is used as super class excepts only fixed set of arguments. Maybe better to pop('cmd") from kwargs to ensure it is not propagated further.

See <https://github.com/jazzband/django-constance/issues/294> for real world issue.


That said, I'd be much happier making the error case the final one. which would explicitly show that we use `EmptyCollectionConstraint` when no special types are recognized and `actual != null`.


This line of command will update my customized index.html page and remove all "/cgi-bin" from it. Is it a by design behavior ?


They are ignored; we could support them, but I don't see any value in writing `contract(fun() { ... })`. And the less constructions we support now, the easier it will be to migrate those declarations to any other syntax (in case we decide to do so)


Close #10


Yup
Due to changes in build.gradle to support Android 9


I'd prefer this to create a temporary file in /tmp. Try the tempfile python module.


Thank for the shoutout! Love what you've done with the project. Keep up the great work!


Parece bom


Thanks for the notice. As far as I understand, they weill be forbidden now, because top-level functions in scripts will have some other containing declarations. I thinks it's fine for now, because I'd like to confirm that it's ok to support contracts here (but generally we'd like to support them).

I've created https://youtrack.jetbrains.com/issue/KT-26288 to not forget that :)


@chelmuth: please add to staging


Ok


I have been working on this already, I dont add an option, but instead rewrite the output of the GETs in the API to not contain capcodes.
Breaking refreshing of the home page is pre big for me.

POST my default is already secure and doesn't require any changing (you need an API key to get any data through)

Mind if I make some changes to this?


Let it bump !


This change has made the calendar list nextmatch too small
![screenshot at 2018-08-21 14-56-29](https://user-images.githubusercontent.com/1624978/44400070-890c8300-a552-11e8-9137-11c3025c3115.png)


hello


Great! Look forward to hearing your thoughts over at #222


Also note that you can modify whether you're using the development operators (built by running the Makefile in the rime_ops directory) or the pip installed operators here:

https://github.com/JoshVStaden/montblanc/blob/dask-tf-1.4/montblanc/impl/rime/tensorflow/tensorflow_ops.py#L18-L25


Hi @gdgellatly. Glad that the care_center modules are getting some port attention, but wondering why you're re-licensing the modules.


[R/Analytics.R:2:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Analytics.R:3:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' \code{Analytics} plays a short sound which is useful if you want to get notified,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:3:85:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L3) *style:* **Trailing whitespace is superfluous.**
```r
#' \code{Analytics} plays a short sound which is useful if you want to get notified,
                                                                                ^
```
[R/Analytics.R:7:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#' If \code{Analytics} is not able to play the sound a warning is issued rather than
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:7:85:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L7) *style:* **Trailing whitespace is superfluous.**
```r
#' If \code{Analytics} is not able to play the sound a warning is issued rather than
                                                                                ^
```
[R/Analytics.R:11:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L11) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sound character string or number specifying what sound to be played by
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:11:81:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#' @param sound character string or number specifying what sound to be played by
                                                                                ^
```
[R/Analytics.R:12:80:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
#'   either specifying one of the built in sounds, specifying the path to a wav
                                                                               ^
```
[R/Analytics.R:16:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L16) *style:* **Lines should not be more than 80 characters.**
```r
#'   or url, a random sound will be played. Currently \code{Analytics} can only handle
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:25:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L25) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Analytics.R:31:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L31) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Analytics.R:34:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L34) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Analytics.R:43:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L43) *style:* **Variable or function name should be snake_case.**
```r
Analytics <- function(sound = 1, expr = NULL) {
^~~~~~~~~
```
[R/Analytics.R:44:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Analytics.R:49:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L49) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Analytics.R:51:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L51) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(sounds[sound]) || length(sounds[sound]) != 1) {
    ^
```
[R/Analytics.R:52:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L52) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.character(sound)) {
      ^
```
[R/Analytics.R:54:9:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L54) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(file.exists(sound)) {
        ^
```
[R/Analytics.R:56:16:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L56) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/Analytics.R:58:16:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L58) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/Analytics.R:59:38:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L59) *style:* **Put spaces around all infix operators.**
```r
        temp_file <- tempfile(pattern="")
                                    ~^~
```
[R/Analytics.R:60:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L60) *style:* **Lines should not be more than 80 characters.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:60:11:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L60) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
          ^
```
[R/Analytics.R:60:75:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L60) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
                                                                          ^
```
[R/Analytics.R:66:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L66) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste('"', sound, '" is not a valid sound nor path, playing a random sound instead.', sep = ""))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:70:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L70) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sounds[sound], sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:70:71:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L70) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sounds[sound], sep=""), package="geekr")
                                                                     ~^~
```
[R/Analytics.R:70:84:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L70) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sounds[sound], sep=""), package="geekr")
                                                                                ~^~
```
[R/Analytics.R:72:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L72) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Analytics.R:73:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L73) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(sound_path)) { # play a random sound
    ^
```
[R/Analytics.R:73:27:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L73) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if(is.null(sound_path)) { # play a random sound
                          ^
```
[R/Analytics.R:74:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L74) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sample(sounds, size=1), sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:74:72:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L74) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sample(sounds, size=1), sep=""), package="geekr")
                                                                      ~^~
```
[R/Analytics.R:74:80:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L74) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sample(sounds, size=1), sep=""), package="geekr")
                                                                              ~^~
```
[R/Analytics.R:74:93:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L74) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sample(sounds, size=1), sep=""), package="geekr")
                                                                                ~^~
```
[R/Analytics.R:76:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L76) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Analytics.R:78:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L78) *style:* **Lines should not be more than 80 characters.**
```r
    warning("Analytics() could not play the sound due to the following error:\n", ex)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:83:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L83) *warning:* **no visible global function definition for ‘str_detect’**
```r
  str_detect(fname, "\\.wav$")
  ^~~~~~~~~~
```
[R/Analytics.R:87:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L87) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:87:116:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L87) *style:* **Trailing whitespace is superfluous.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
                                                                                ^
```
[R/Analytics.R:88:45:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L88) *style:* **Put spaces around all infix operators.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                           ~^~
```
[R/Analytics.R:88:51:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L88) *style:* **Commas should always have a space after.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                  ^
```
[R/Analytics.R:93:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L93) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:93:70:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L93) *style:* **Put spaces around all infix operators.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                    ~^~
```
[R/Analytics.R:93:76:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L93) *style:* **Commas should always have a space after.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                           ^
```
[R/Analytics.R:98:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L98) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:98:94:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L98) *style:* **Put spaces around all infix operators.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ~^~
```
[R/Analytics.R:98:100:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L98) *style:* **Commas should always have a space after.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ^
```
[R/Analytics.R:103:10:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L103) *warning:* **no visible global function definition for ‘load.wave’**
```r
  sfx <- load.wave(fname)
         ^~~~~~~~~
```
[R/Analytics.R:104:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L104) *warning:* **no visible global function definition for ‘play’, Did you mean 'lag'?**
```r
  play(sfx)
  ^~~~
```
[R/Analytics.R:108:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L108) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(Sys.info()["sysname"] == "Linux") {
    ^
```
[R/Analytics.R:109:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L109) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is_wav_fname(fname) && nchar(Sys.which("paplay")) >= 1) {
      ^
```
[R/Analytics.R:110:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L110) *warning:* **no visible global function definition for ‘play_paplay’**
```r
      play_paplay(fname)
      ^~~~~~~~~~~
```
[R/Analytics.R:111:14:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L111) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(is_wav_fname(fname) && nchar(Sys.which("aplay")) >= 1) {
             ^
```
[R/Analytics.R:112:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L112) *warning:* **no visible global function definition for ‘play_aplay’**
```r
      play_aplay(fname)
      ^~~~~~~~~~
```
[R/Analytics.R:113:14:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L113) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(nchar(Sys.which("vlc")) >= 1) {
             ^
```
[R/Analytics.R:114:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L114) *warning:* **no visible global function definition for ‘play_vlc’**
```r
      play_vlc(fname)
      ^~~~~~~~
```
[R/Analytics.R:116:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L116) *warning:* **no visible global function definition for ‘play_audio’**
```r
      play_audio(fname)
      ^~~~~~~~~~
```
[R/Analytics.R:119:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Analytics.R#L119) *warning:* **no visible global function definition for ‘play_audio’**
```r
    play_audio(fname)
    ^~~~~~~~~~
```
[R/ChronoTrigger.R:2:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/ChronoTrigger.R:3:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' \code{ChronoTrigger} plays a short sound which is useful if you want to get notified,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:3:89:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L3) *style:* **Trailing whitespace is superfluous.**
```r
#' \code{ChronoTrigger} plays a short sound which is useful if you want to get notified,
                                                                                ^
```
[R/ChronoTrigger.R:7:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#' If \code{ChronoTrigger} is not able to play the sound a warning is issued rather than
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:7:89:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L7) *style:* **Trailing whitespace is superfluous.**
```r
#' If \code{ChronoTrigger} is not able to play the sound a warning is issued rather than
                                                                                ^
```
[R/ChronoTrigger.R:11:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L11) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sound character string or number specifying what sound to be played by
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:11:81:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#' @param sound character string or number specifying what sound to be played by
                                                                                ^
```
[R/ChronoTrigger.R:12:80:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
#'   either specifying one of the built in sounds, specifying the path to a wav
                                                                               ^
```
[R/ChronoTrigger.R:18:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L18) *style:* **Lines should not be more than 80 characters.**
```r
#'   \code{"black_tyrano"} \item \code{"continuum"} \item \code{"destruction"} \item
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:22:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L22) *style:* **Lines should not be more than 80 characters.**
```r
#'   or url, a random sound will be played. Currently \code{ChronoTrigger} can only handle
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:31:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L31) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/ChronoTrigger.R:37:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L37) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/ChronoTrigger.R:40:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L40) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/ChronoTrigger.R:49:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L49) *style:* **Variable or function name should be snake_case.**
```r
ChronoTrigger <- function(sound = 1, expr = NULL) {
^~~~~~~~~~~~~
```
[R/ChronoTrigger.R:50:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L50) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/ChronoTrigger.R:72:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L72) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/ChronoTrigger.R:74:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L74) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(sounds[sound]) || length(sounds[sound]) != 1) {
    ^
```
[R/ChronoTrigger.R:75:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L75) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.character(sound)) {
      ^
```
[R/ChronoTrigger.R:77:9:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L77) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(file.exists(sound)) {
        ^
```
[R/ChronoTrigger.R:79:16:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L79) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/ChronoTrigger.R:81:16:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L81) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/ChronoTrigger.R:82:38:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L82) *style:* **Put spaces around all infix operators.**
```r
        temp_file <- tempfile(pattern="")
                                    ~^~
```
[R/ChronoTrigger.R:83:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L83) *style:* **Lines should not be more than 80 characters.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:83:11:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L83) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
          ^
```
[R/ChronoTrigger.R:83:75:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L83) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
                                                                          ^
```
[R/ChronoTrigger.R:89:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L89) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste('"', sound, '" is not a valid sound nor path, playing a random sound instead.', sep = ""))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:93:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L93) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/CT/", sounds[sound], sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:93:69:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L93) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sounds[sound], sep=""), package="geekr")
                                                                   ~^~
```
[R/ChronoTrigger.R:93:82:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L93) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sounds[sound], sep=""), package="geekr")
                                                                                ~^~
```
[R/ChronoTrigger.R:95:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L95) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/ChronoTrigger.R:96:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L96) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(sound_path)) { # play a random sound
    ^
```
[R/ChronoTrigger.R:96:27:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L96) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if(is.null(sound_path)) { # play a random sound
                          ^
```
[R/ChronoTrigger.R:97:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L97) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/CT/", sample(sounds, size=1), sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:97:70:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L97) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sample(sounds, size=1), sep=""), package="geekr")
                                                                    ~^~
```
[R/ChronoTrigger.R:97:78:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L97) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sample(sounds, size=1), sep=""), package="geekr")
                                                                            ~^~
```
[R/ChronoTrigger.R:97:91:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L97) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sample(sounds, size=1), sep=""), package="geekr")
                                                                                ~^~
```
[R/ChronoTrigger.R:99:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L99) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/ChronoTrigger.R:101:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L101) *style:* **Lines should not be more than 80 characters.**
```r
    warning("ChronoTrigger() could not play the sound due to the following error:\n", ex)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:106:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L106) *warning:* **no visible global function definition for ‘str_detect’**
```r
  str_detect(fname, "\\.wav$")
  ^~~~~~~~~~
```
[R/ChronoTrigger.R:110:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L110) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:110:116:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L110) *style:* **Trailing whitespace is superfluous.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
                                                                                ^
```
[R/ChronoTrigger.R:111:45:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L111) *style:* **Put spaces around all infix operators.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                           ~^~
```
[R/ChronoTrigger.R:111:51:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L111) *style:* **Commas should always have a space after.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                  ^
```
[R/ChronoTrigger.R:116:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L116) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:116:70:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L116) *style:* **Put spaces around all infix operators.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                    ~^~
```
[R/ChronoTrigger.R:116:76:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L116) *style:* **Commas should always have a space after.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                           ^
```
[R/ChronoTrigger.R:121:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L121) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:121:94:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L121) *style:* **Put spaces around all infix operators.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ~^~
```
[R/ChronoTrigger.R:121:100:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L121) *style:* **Commas should always have a space after.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ^
```
[R/ChronoTrigger.R:126:10:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L126) *warning:* **no visible global function definition for ‘load.wave’**
```r
  sfx <- load.wave(fname)
         ^~~~~~~~~
```
[R/ChronoTrigger.R:127:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L127) *warning:* **no visible global function definition for ‘play’, Did you mean 'lag'?**
```r
  play(sfx)
  ^~~~
```
[R/ChronoTrigger.R:131:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L131) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(Sys.info()["sysname"] == "Linux") {
    ^
```
[R/ChronoTrigger.R:132:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L132) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is_wav_fname(fname) && nchar(Sys.which("paplay")) >= 1) {
      ^
```
[R/ChronoTrigger.R:133:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L133) *warning:* **no visible global function definition for ‘play_paplay’**
```r
      play_paplay(fname)
      ^~~~~~~~~~~
```
[R/ChronoTrigger.R:134:14:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L134) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(is_wav_fname(fname) && nchar(Sys.which("aplay")) >= 1) {
             ^
```
[R/ChronoTrigger.R:135:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L135) *warning:* **no visible global function definition for ‘play_aplay’**
```r
      play_aplay(fname)
      ^~~~~~~~~~
```
[R/ChronoTrigger.R:136:14:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L136) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(nchar(Sys.which("vlc")) >= 1) {
             ^
```
[R/ChronoTrigger.R:137:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L137) *warning:* **no visible global function definition for ‘play_vlc’**
```r
      play_vlc(fname)
      ^~~~~~~~
```
[R/ChronoTrigger.R:139:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L139) *warning:* **no visible global function definition for ‘play_audio’**
```r
      play_audio(fname)
      ^~~~~~~~~~
```
[R/ChronoTrigger.R:142:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/ChronoTrigger.R#L142) *warning:* **no visible global function definition for ‘play_audio’**
```r
    play_audio(fname)
    ^~~~~~~~~~
```
[R/SuperMario.R:2:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/SuperMario.R:3:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' \code{SuperMario} plays a short sound which is useful if you want to get notified,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:3:86:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L3) *style:* **Trailing whitespace is superfluous.**
```r
#' \code{SuperMario} plays a short sound which is useful if you want to get notified,
                                                                                ^
```
[R/SuperMario.R:7:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#' If \code{SuperMario} is not able to play the sound a warning is issued rather than
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:7:86:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L7) *style:* **Trailing whitespace is superfluous.**
```r
#' If \code{SuperMario} is not able to play the sound a warning is issued rather than
                                                                                ^
```
[R/SuperMario.R:11:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L11) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sound character string or number specifying what sound to be played by
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:11:81:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#' @param sound character string or number specifying what sound to be played by
                                                                                ^
```
[R/SuperMario.R:12:80:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
#'   either specifying one of the built in sounds, specifying the path to a wav
                                                                               ^
```
[R/SuperMario.R:23:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L23) *style:* **Lines should not be more than 80 characters.**
```r
#'   or url, a random sound will be played. Currently \code{SuperMario} can only handle
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:32:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L32) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/SuperMario.R:38:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L38) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/SuperMario.R:41:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L41) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/SuperMario.R:50:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L50) *style:* **Variable or function name should be snake_case.**
```r
SuperMario <- function(sound = 1, expr = NULL) {
^~~~~~~~~~
```
[R/SuperMario.R:51:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L51) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/SuperMario.R:75:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L75) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/SuperMario.R:77:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L77) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(sounds[sound]) || length(sounds[sound]) != 1) {
    ^
```
[R/SuperMario.R:78:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L78) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.character(sound)) {
      ^
```
[R/SuperMario.R:80:9:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L80) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(file.exists(sound)) {
        ^
```
[R/SuperMario.R:82:16:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L82) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/SuperMario.R:84:16:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L84) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/SuperMario.R:85:38:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L85) *style:* **Put spaces around all infix operators.**
```r
        temp_file <- tempfile(pattern="")
                                    ~^~
```
[R/SuperMario.R:86:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L86) *style:* **Lines should not be more than 80 characters.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:86:11:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L86) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
          ^
```
[R/SuperMario.R:86:75:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L86) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
                                                                          ^
```
[R/SuperMario.R:92:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L92) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste('"', sound, '" is not a valid sound nor path, playing a random sound instead.', sep = ""))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:96:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L96) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sounds[sound], sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:96:70:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L96) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sounds[sound], sep=""), package="geekr")
                                                                    ~^~
```
[R/SuperMario.R:96:83:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L96) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sounds[sound], sep=""), package="geekr")
                                                                                ~^~
```
[R/SuperMario.R:98:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L98) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/SuperMario.R:99:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L99) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(sound_path)) { # play a random sound
    ^
```
[R/SuperMario.R:99:27:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L99) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if(is.null(sound_path)) { # play a random sound
                          ^
```
[R/SuperMario.R:100:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L100) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sample(sounds, size=1), sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:100:71:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L100) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sample(sounds, size=1), sep=""), package="geekr")
                                                                     ~^~
```
[R/SuperMario.R:100:79:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L100) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sample(sounds, size=1), sep=""), package="geekr")
                                                                             ~^~
```
[R/SuperMario.R:100:92:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L100) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sample(sounds, size=1), sep=""), package="geekr")
                                                                                ~^~
```
[R/SuperMario.R:102:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L102) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/SuperMario.R:104:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L104) *style:* **Lines should not be more than 80 characters.**
```r
    warning("SuperMario() could not play the sound due to the following error:\n", ex)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:109:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L109) *warning:* **no visible global function definition for ‘str_detect’**
```r
  str_detect(fname, "\\.wav$")
  ^~~~~~~~~~
```
[R/SuperMario.R:113:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L113) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:113:116:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L113) *style:* **Trailing whitespace is superfluous.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
                                                                                ^
```
[R/SuperMario.R:114:45:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L114) *style:* **Put spaces around all infix operators.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                           ~^~
```
[R/SuperMario.R:114:51:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L114) *style:* **Commas should always have a space after.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                  ^
```
[R/SuperMario.R:119:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L119) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:119:70:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L119) *style:* **Put spaces around all infix operators.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                    ~^~
```
[R/SuperMario.R:119:76:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L119) *style:* **Commas should always have a space after.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                           ^
```
[R/SuperMario.R:124:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L124) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:124:94:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L124) *style:* **Put spaces around all infix operators.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ~^~
```
[R/SuperMario.R:124:100:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L124) *style:* **Commas should always have a space after.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ^
```
[R/SuperMario.R:129:10:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L129) *warning:* **no visible global function definition for ‘load.wave’**
```r
  sfx <- load.wave(fname)
         ^~~~~~~~~
```
[R/SuperMario.R:130:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L130) *warning:* **no visible global function definition for ‘play’, Did you mean 'lag'?**
```r
  play(sfx)
  ^~~~
```
[R/SuperMario.R:134:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L134) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(Sys.info()["sysname"] == "Linux") {
    ^
```
[R/SuperMario.R:135:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L135) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is_wav_fname(fname) && nchar(Sys.which("paplay")) >= 1) {
      ^
```
[R/SuperMario.R:136:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L136) *warning:* **no visible global function definition for ‘play_paplay’**
```r
      play_paplay(fname)
      ^~~~~~~~~~~
```
[R/SuperMario.R:137:14:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L137) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(is_wav_fname(fname) && nchar(Sys.which("aplay")) >= 1) {
             ^
```
[R/SuperMario.R:138:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L138) *warning:* **no visible global function definition for ‘play_aplay’**
```r
      play_aplay(fname)
      ^~~~~~~~~~
```
[R/SuperMario.R:139:14:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L139) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(nchar(Sys.which("vlc")) >= 1) {
             ^
```
[R/SuperMario.R:140:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L140) *warning:* **no visible global function definition for ‘play_vlc’**
```r
      play_vlc(fname)
      ^~~~~~~~
```
[R/SuperMario.R:142:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L142) *warning:* **no visible global function definition for ‘play_audio’**
```r
      play_audio(fname)
      ^~~~~~~~~~
```
[R/SuperMario.R:145:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/SuperMario.R#L145) *warning:* **no visible global function definition for ‘play_audio’**
```r
    play_audio(fname)
    ^~~~~~~~~~
```
[R/Warcraft.R:2:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Warcraft.R:3:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' \code{Warcraft} plays a short sound which is useful if you want to get notified,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:3:84:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L3) *style:* **Trailing whitespace is superfluous.**
```r
#' \code{Warcraft} plays a short sound which is useful if you want to get notified,
                                                                                ^
```
[R/Warcraft.R:7:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#' If \code{Warcraft} is not able to play the sound a warning is issued rather than
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:7:84:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L7) *style:* **Trailing whitespace is superfluous.**
```r
#' If \code{Warcraft} is not able to play the sound a warning is issued rather than
                                                                                ^
```
[R/Warcraft.R:11:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L11) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sound character string or number specifying what sound to be played by
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:11:81:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#' @param sound character string or number specifying what sound to be played by
                                                                                ^
```
[R/Warcraft.R:12:80:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
#'   either specifying one of the built in sounds, specifying the path to a wav
                                                                               ^
```
[R/Warcraft.R:19:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L19) *style:* **Lines should not be more than 80 characters.**
```r
#'   or url, a random sound will be played. Currently \code{Warcraft} can only handle
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:28:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L28) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Warcraft.R:34:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L34) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Warcraft.R:37:3:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L37) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Warcraft.R:46:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L46) *style:* **Variable or function name should be snake_case.**
```r
Warcraft <- function(sound = 1, expr = NULL) {
^~~~~~~~
```
[R/Warcraft.R:47:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L47) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Warcraft.R:61:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L61) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Warcraft.R:63:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L63) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(sounds[sound]) || length(sounds[sound]) != 1) {
    ^
```
[R/Warcraft.R:64:7:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L64) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.character(sound)) {
      ^
```
[R/Warcraft.R:66:9:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L66) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(file.exists(sound)) {
        ^
```
[R/Warcraft.R:68:16:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L68) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/Warcraft.R:70:16:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L70) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/Warcraft.R:71:38:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L71) *style:* **Put spaces around all infix operators.**
```r
        temp_file <- tempfile(pattern="")
                                    ~^~
```
[R/Warcraft.R:72:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L72) *style:* **Lines should not be more than 80 characters.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:72:11:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L72) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
          ^
```
[R/Warcraft.R:72:75:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L72) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
                                                                          ^
```
[R/Warcraft.R:78:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L78) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste('"', sound, '" is not a valid sound nor path, playing a random sound instead.', sep = ""))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:82:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L82) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/Warcraft/", sounds[sound], sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:82:75:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L82) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/Warcraft/", sounds[sound], sep=""), package="geekr")
                                                                         ~^~
```
[R/Warcraft.R:82:88:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L82) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/Warcraft/", sounds[sound], sep=""), package="geekr")
                                                                                ~^~
```
[R/Warcraft.R:84:1:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L84) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Warcraft.R:85:5:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L85) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(sound_path)) { # play a random sound
    ^
```
[R/Warcraft.R:85:27:](https://github.com/mike-decarlo/geekr/blob/ed9bd5b4fd5c7f4b5abe911ce06cb006909ebffc/R/Warcraft.R#L85) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if(is.null(sound_path)) { # play a random sound
                          ^
```


Go for it ;-)

I was more building this towards this kind of issue https://github.com/davidmckenzie/pagermon/issues/77 where the user doesn't want people seeing ***anything***

If your filtering out the capcodes - that may be something that toggles with the options from my DisplayFiltering Pull Request?


The reason why `six.b` was not working was that the module was not imported.


That's unfortunate. I'd be happy to work with you on Discord for 15 min or so. Something silly us going on


Also i wouldn't have asked for opinions if i didn't want to be told i'm doing it wrong.... 👍

I've literally never worked with node/angular/ejs/whatever the hell this is built in in my life haha


git clone https://github.com/ethereum/remix-ide.git
cd remix-ide
npm install
npm run build && npm run serve


这代码写的很好


probably replace by a select(uom robustness) and a orm unlink


Sure, they'll need their own instructions in `DOCKER-INSTALL.md` since just running `npm test` won't work for them. 🙃


Fixes #5.


Glad for the porting work, but why the sweeping lint & license changes? Changing a single quotes to double quotes seems especially gratuitous - that serves no useful purpose whatsoever and it makes unnecessary code diffs.


I think it's better to check for syntax errors instead of  flock,   and get rid of unnecessary checks file_exists.


... which they already seem to have: https://github.com/Trustroots/trustroots/blob/2c54afc8501877ee1491b8ef052ecada1293e587/INSTALL-DOCKER.md#running-tests


This kind of extra whitespacing makes it slightly harder to read the domain and not easier. Of course it could be subject to user preference, but why doesn't my preference as original author count for something?


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


I wrote a fix for that person directly, looks to be working well.

Your doing some good stuff for never played with it before, I was in the same boat 12 months ago 😂


@feld-ims @ceagan https://github.com/openshift/openshift-ansible/pull/9679 should've fixed this on release-3.9 branch, are you guys working on this branch or release-3.10?


@thinkwelltwd I meant to email you about license actually. I forgot.  The LICENSE file in repo is LGPL, so I thought it was meant to all be LGPL, but then I noticed github had inserted LGPL file into my own repos so was like, maybe it is github not thinkwell.

As for the quotes, it is just autolint using black.


Remove


我觉得不行


Thanks, merged all four commits to staging.


The whole function should be wrap with locks. (`file_put_contents`  and `opcache_compile_file`  must be in the same critical section)


仔细看看还好


Hmm, this behaviour seems a bit inconsistent with other parsers... But let's keep it, at least for now.


delete


Minor: `left is ConstantReference` looks redundant. Same below.


bak


Is this based in any way on the 5.7 percona_statement_set.test?


please use consistent SQL keyword case


I approve.


Généralement le titre du commit doit être à la forme active (`Add V1.2 of the chandelay module`)

Mais en l'occurence ce commit doit être squash avec le commit qui rajoute les sources


--source include/count_sessions.inc somewhere above, wait_until_count_sessions.inc somewhere at the end


Sorry for me pointing out the obvious. I understand that elementaryOS is your primary target. It's just sad that ubuntu's patched stuff is the "default" here, and not the "vanilla" version - but that's ubuntu's "fault", and not yours.


This might be problematic from user experience point of view: I'm typing something inside `contract { ... }`, resolve starts, finds an unresolved name, all code turns red, aaawwwww...


Thanks 😊

There was also an issue raised by this guy for something similar https://github.com/davidmckenzie/pagermon/issues/39

Haha :grin:


SystemLocale dsc module was generating errors when used.
Module SystemLocaleDsc.ResourceHelper.psm1 could not be found in SystemLocaleDsc/Modules/SystemLocaleDsc.ResourceHelper/SystemLocaleDsc.ResourceHelper.psm1.

Folder /SystemLocaleDsc.ResourceHelper/SystemLocaleDsc.ResourceHelper.psm1 moved tot Modules folder under SystemLocaleDsc and now it works just fine


Stocke juste un vlc_mutex_t (au lieu d'un pointeur) dans p_sys, et passe son adresse aux fonction vlc_mutex_*, ça t'évite de devoir malloc/free le mutex


Le mutex est toujours locké si tu quittes la fonction ici


Is it possible to report `CONTRACT_NOT_ALLOWED` on `contract`?


added new line and committed


You might want to consider using the `<textarea>` tag instead of `<input>` for the "comments" section https://www.w3schools.com/tags/tag_textarea.asp


Actually what are the semantics then? Are the invalid values ignored?


Sorry. You're right. The master is on 2.27.
This means 1.3.6 will have it.
Jersey has some problems with backward compatibility from 2.25 to 2.26+
You won't be able to keep using the current jersey2-guice version, which is 3 years old and will not support jersey 2.27 as is.
I worked on a solution for this. You can see my experiment here: https://github.com/next-insurance/jersey2-guice


s/NULL/nullptr/g


I was on the 3.9 branch, I applied the committed fix manually and it did indeed resolve the problem


你再仔细看看？


You should probably add some styling to the "submit" button, especially for mobile since it gets a little hard to see then


Please set up the git clean/smudge filter with clang-format 5.0 :)


Sauf erreur, le format peut changer en cours d'utilisation (à vérifier quand même), mais du coup je serais plus d'avis de toujours utiliser le format d'input au lieu d'une version cachée


How did this work?


jsut do it


你再仔细看看啊


const method?


Tu peux utiliser un tableau static au lieu d'un tableau heap-allocated


If actual is a value type, then it is not legitimate to compare it with null.  So perhaps there needs to be additional tests.

It appears that ImmutableArray was a class? (`public static class ImmutableArray`)


The code for the program goes in this file. (Collaborators : @swapniel5 )


This may not work as this section of the indexing configuration file has to be exactly the same as what comes out of the `prepare_metadata` cli (for `datacube dataset add` to automatically match metadata with already registered products). My advice would be to forget about landsat 4. Too few data, and we're not planning to go that far in the past at the moment.


If you use the tempfile module as suggested above, you should not remove the file until the tearDown method (or whatever it's supposed to be called) is called.


see new branch I have created.