Yikes, glad you pointed this out, I wasn't recalling this about Ruby.  Interestingly, the change works in the Rakefile, but not in a .rb file.

Here's a quick test module  :
```
$ cat test.rb
@a = 1
b = 2
# Globals are bad
$c = "3 -- Dangerous"
d = 4

def func1()
  puts(@a)
end

define_method :func2 do
  puts b
end

def func3()
  puts($c)
end

def func4()
  puts d
end

func1()
func2()
func3()
func4()
```
Executing the test
```
$ ruby test.rb
1
2
3 -- Dangerous
test.rb:20:in `func4': undefined local variable or method `d' for main:Object (NameError)
	from test.rb:26:in `<main>'

```
Numbered per variables used above:
1) Instance variables are an option: http://stackoverflow.com/a/5890199/320399
2) Using `define_method` won't create a new scope, so that's an option: http://stackoverflow.com/a/10782889/320399
3) I agree, globals are bad.
4) Fails, as you pointed out.

This small discussion might be a useful Ruby refresher for the team.


No need to pass in length.. can get from array.length


Nice use of filter.


You're welcome!


i've done thousand of screenshots.


Change the `CHILDS` to `CHILDREN` here. (you can keep `childAccounts` elsewhere)


Instead: `rootProject.file("build-changelog.xml")`


Doesn't this break the compilation of the paper (i.e., in paper/ directory)? I thought these markers are used for extracting some code for the paper. (but please do correct me if I'm wrong or missing something!)

Even if they are used, it's IMHO a fair question whether we want to maintain the paper or not? But that should be a deliberate decision (in contrast, removing the unused DataflowFold and OldDataflow seems uncontroversial, at least to me)


:bug: huh? What tests fail?

I think what you meant to do was override `isDefault()` to return true, since the default implementation uses object identity rather than by comparing `getViewName()`. (Arguably a bug in Jenkins core.)


@jglick *you* told me to!


Moved to a separate method and as discussed made use of GobblinMetrics to use its Reporters


short answer: no. The reason I coded this way was if I got more than one than test should fail.


If we're talking about a new release, let's add a ton of debug statements.


ahhh this is so good


@jglick  ajaxBuildQueue returns 404, as it's relative to the .../view/


@mkawalec powodzenia w tworzeniu swojego bloga! ;)


In future, keep formatting changes, like spacing changes throughout a module etc, as isolated commits.


Done.


No message was being sent this time. The previous message was a remainder from the previous versions of this guide. :)


Unfortunately that is not in-line with (#459). Will probably go away.


I wanted to see if I could make the name checking logic cleaner, and here's what I came up with. I'm not sure whether it's a significant improvement, but I'm posting it anyway for consideration.

```go
repoName := pd.repoInfo.RemoteName()
sourceRepoName := pd.repoInfo.RemoteName()

repoLastSlash := strings.LastIndex(repoName, "/")
sourceRepoLastSlash := strings.LastIndex(sourceRepoName, "/")

// validate that both repositories have the same name components before the last component
if repoLastSlash != sourceRepoLastSlash {
        continue
}
if repoName[:repoLastSlash+1] != sourceRepoName[:sourceRepoLastSlash+1] {
        continue
}

// validate that both repositories have different last name components
if repoName[repoLastSlash+1:] == sourceRepoName[sourceRepoLastSlash+1:] {
        continue
}
```


Here is says that `isCanceled()` can be used in `onFailure`. `onFailure` is a method in `Callback` interface. We don't always have the reference to `Call` object inside `Callback`. What is the best way to detect the cancellation?


pocketmine\block\Level Who the hell wrote this


It doesn't promise anything, but it doesn't do as well :)


It doesn't look like you are using `requestsDB`.


I merged everything in one commit.


Yeah, instance variables might be ok in this context.

In the Rakefile, task...end is not the same as def...end. So you are still in the same scope, so you can refer to stuff defined outside the task..end block. That's why it works there, but I was importing habits that were not appropriate in this context.


pocketmine\level\Level ?


[R/boot_jaatha.R:24:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/boot_jaatha.R#L24) *warning:* **no visible global function definition for ‘require_package’**
```r
  require_package("boot")
  ^~~~~~~~~~~~~~~
```
[R/boot_jaatha.R:49:5:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/boot_jaatha.R#L49) *warning:* **no visible global function definition for ‘create_jaatha_data.default’**
```r
    create_jaatha_data.default(sim_data, model)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/coala_interface.R:45:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/coala_interface.R#L45) *warning:* **no visible global function definition for ‘require_package’**
```r
  require_package("coala")
  ^~~~~~~~~~~~~~~
```
[R/coala_interface.R:70:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/coala_interface.R#L70) *warning:* **no visible global function definition for ‘require_package’**
```r
  require_package("coala")
  ^~~~~~~~~~~~~~~
```
[R/coala_interface.R:71:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/coala_interface.R#L71) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.string(jsfs_summary))
  ^~~~~~~~~~~
```
[R/coala_interface.R:71:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/coala_interface.R#L71) *warning:* **no visible global function definition for ‘is.string’**
```r
  assert_that(is.string(jsfs_summary))
              ^~~~~~~~~
```
[R/coala_interface.R:80:11:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/coala_interface.R#L80) *warning:* **no visible global function definition for ‘coarsen_jsfs’**
```r
          coarsen_jsfs(x[[name]], jsfs_part, jsfs_part_hi)
          ^~~~~~~~~~~~
```
[R/coala_interface.R:93:14:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/coala_interface.R#L93) *warning:* **no visible global function definition for ‘create_jaatha_stat’**
```r
      return(create_jaatha_stat(name, function(x, opts) x[[name]]))
             ^~~~~~~~~~~~~~~~~~
```
[R/coala_interface.R:199:20:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/coala_interface.R#L199) *warning:* **no visible global function definition for ‘multi_index_range’**
```r
    z[i] <- sum(ja[multi_index_range(d, p)])
                   ^~~~~~~~~~~~~~~~~
```
[R/fit_glm.R:10:29:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/fit_glm.R#L10) *warning:* **no visible binding for global variable ‘fit_glm’**
```r
  lapply(x$get_sum_stats(), fit_glm, sim_data, ...)
                            ^~~~~~~
```
[R/initialization.R:6:18:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L6) *warning:* **no visible global function definition for ‘do_zoom_in_search’**
```r
    start_pos <- do_zoom_in_search(model, data, reps, sim, cores, sim_cache)
                 ^~~~~~~~~~~~~~~~~
```
[R/initialization.R:8:18:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L8) *warning:* **no visible global function definition for ‘do_initial_search’**
```r
    start_pos <- do_initial_search(model, data, reps, sim, cores, sim_cache)
                 ^~~~~~~~~~~~~~~~~
```
[R/initialization.R:18:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L18) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.matrix(start_pos))
  ^~~~~~~~~~~
```
[R/initialization.R:19:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L19) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(all(dim(start_pos) == c(reps, model$get_par_number())))
  ^~~~~~~~~~~
```
[R/initialization.R:27:21:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L27) *warning:* **no visible global function definition for ‘determine_bpp’**
```r
  blocks_per_par <- determine_bpp(par_number, reps)
                    ^~~~~~~~~~~~~
```
[R/initialization.R:28:13:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L28) *warning:* **no visible global function definition for ‘create_initial_blocks’**
```r
  blocks <- create_initial_blocks(model$get_par_ranges(), blocks_per_par)
            ^~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:62:17:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L62) *warning:* **no visible global function definition for ‘create_block’**
```r
    return(list(create_block(basic_block)))
                ^~~~~~~~~~~~
```
[R/initialization.R:72:45:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L72) *warning:* **no visible global function definition for ‘create_block’**
```r
      blocks[[(j - 1) * par_number + i]] <- create_block(new_block)
                                            ^~~~~~~~~~~~
```
[R/initialization.R:86:17:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/initialization.R#L86) *warning:* **no visible global function definition for ‘estimate_local_ml’**
```r
      middle <- estimate_local_ml(block, model, data, sim, cores, sim_cache)$par
                ^~~~~~~~~~~~~~~~~
```
[R/jaatha_data.R:84:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha_data.R#L84) *warning:* **no visible binding for global variable ‘jaatha_data_class’**
```r
  jaatha_data_class$new(data, model)
  ^~~~~~~~~~~~~~~~~
```
[R/jaatha_data.R:90:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha_data.R#L90) *warning:* **no visible global function definition for ‘create_jaatha_data’**
```r
  create_jaatha_data(test_data, model)
  ^~~~~~~~~~~~~~~~~~
```
[R/jaatha.R:76:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L76) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_model(model))
  ^~~~~~~~~~~
```
[R/jaatha.R:76:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L76) *warning:* **no visible global function definition for ‘is_jaatha_model’**
```r
  assert_that(is_jaatha_model(model))
              ^~~~~~~~~~~~~~~
```
[R/jaatha.R:77:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L77) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_data(data))
  ^~~~~~~~~~~
```
[R/jaatha.R:77:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L77) *warning:* **no visible global function definition for ‘is_jaatha_data’**
```r
  assert_that(is_jaatha_data(data))
              ^~~~~~~~~~~~~~
```
[R/jaatha.R:78:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L78) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(repetitions))
  ^~~~~~~~~~~
```
[R/jaatha.R:78:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L78) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(repetitions))
              ^~~~~~~~
```
[R/jaatha.R:79:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L79) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(sim))
  ^~~~~~~~~~~
```
[R/jaatha.R:79:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L79) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(sim))
              ^~~~~~~~
```
[R/jaatha.R:80:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L80) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(cores))
  ^~~~~~~~~~~
```
[R/jaatha.R:80:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L80) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(cores))
              ^~~~~~~~
```
[R/jaatha.R:81:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L81) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.numeric(block_width) && length(block_width) == 1)
  ^~~~~~~~~~~
```
[R/jaatha.R:82:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L82) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(block_width > 0 && block_width < 1)
  ^~~~~~~~~~~
```
[R/jaatha.R:85:10:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L85) *warning:* **no visible global function definition for ‘create_jaatha_log’**
```r
  log <- create_jaatha_log(model, data, repetitions, max_steps, verbose = TRUE)
         ^~~~~~~~~~~~~~~~~
```
[R/jaatha.R:106:19:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/jaatha.R#L106) *warning:* **no visible global function definition for ‘estimate_local_ml’**
```r
      local_ml <- estimate_local_ml(block, model, data, sim, cores, sim_cache)
                  ^~~~~~~~~~~~~~~~~
```
[R/likelihood.R:14:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L14) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.finite(llh))
  ^~~~~~~~~~~
```
[R/likelihood.R:32:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L32) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_data(data))
  ^~~~~~~~~~~
```
[R/likelihood.R:32:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L32) *warning:* **no visible global function definition for ‘is_jaatha_data’**
```r
  assert_that(is_jaatha_data(data))
              ^~~~~~~~~~~~~~
```
[R/likelihood.R:33:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L33) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.numeric(param))
  ^~~~~~~~~~~
```
[R/likelihood.R:34:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L34) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.list(glm_fitted))
  ^~~~~~~~~~~
```
[R/likelihood.R:42:1:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L42) *style:* **Variable and function names should not be longer than 30 characters.**
```r
approximate_llh.jaatha_stat_basic  <- function(x, data, param, glm_fitted,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/likelihood.R:50:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L50) *warning:* **no visible global function definition for ‘calc_poisson_llh’**
```r
  calc_poisson_llh(data, x, loglambda, sim, scaling_factor)
  ^~~~~~~~~~~~~~~~
```
[R/likelihood.R:59:25:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L59) *warning:* **no visible global function definition for ‘approximate_llh’**
```r
                        approximate_llh(model, data, param, glms, sim)
                        ^~~~~~~~~~~~~~~
```
[R/likelihood.R:66:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L66) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(block$includes(best_value$par))
  ^~~~~~~~~~~
```
[R/likelihood.R:79:5:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L79) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(sim_data) >= sim)
    ^~~~~~~~~~~
```
[R/likelihood.R:82:22:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L82) *warning:* **no visible global function definition for ‘fit_glm’**
```r
    glms <- tryCatch(fit_glm(model, sim_data), error = identity)
                     ^~~~~~~
```
[R/likelihood.R:92:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L92) *warning:* **no visible global function definition for ‘optimize_llh’**
```r
  optimize_llh(block, model, data, glms, length(sim_data))
  ^~~~~~~~~~~~
```
[R/likelihood.R:118:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L118) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_model(model))
  ^~~~~~~~~~~
```
[R/likelihood.R:118:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L118) *warning:* **no visible global function definition for ‘is_jaatha_model’**
```r
  assert_that(is_jaatha_model(model))
              ^~~~~~~~~~~~~~~
```
[R/likelihood.R:119:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L119) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_data(data))
  ^~~~~~~~~~~
```
[R/likelihood.R:119:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L119) *warning:* **no visible global function definition for ‘is_jaatha_data’**
```r
  assert_that(is_jaatha_data(data))
              ^~~~~~~~~~~~~~
```
[R/likelihood.R:120:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L120) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.numeric(parameter))
  ^~~~~~~~~~~
```
[R/likelihood.R:121:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L121) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(sim))
  ^~~~~~~~~~~
```
[R/likelihood.R:121:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L121) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(sim))
              ^~~~~~~~
```
[R/likelihood.R:122:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L122) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(cores))
  ^~~~~~~~~~~
```
[R/likelihood.R:122:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L122) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(cores))
              ^~~~~~~~
```
[R/likelihood.R:123:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L123) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_single_logical(normalized))
  ^~~~~~~~~~~
```
[R/likelihood.R:123:15:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L123) *warning:* **no visible global function definition for ‘is_single_logical’**
```r
  assert_that(is_single_logical(normalized))
              ^~~~~~~~~~~~~~~~~
```
[R/likelihood.R:135:5:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L135) *warning:* **no visible global function definition for ‘calc_poisson_llh’**
```r
    calc_poisson_llh(data, stat, log_means, sim, model$get_scaling_factor())
    ^~~~~~~~~~~~~~~~
```
[R/likelihood.R:138:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/likelihood.R#L138) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.finite(llh))
  ^~~~~~~~~~~
```
[R/stat_basic.R:62:23:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/stat_basic.R#L62) *warning:* **no visible binding for global variable ‘stat_basic_class’**
```r
  if (poisson) return(stat_basic_class$new(name, calc_func))
                      ^~~~~~~~~~~~~~~~
```
[R/stat_basic.R:63:3:](https://github.com/statgenlmu/jaatha/blob/2f228f73e6bba9885629aa0badab4e5838d14e06/R/stat_basic.R#L63) *warning:* **no visible binding for global variable ‘stat_cube_class’**
```r
  stat_cube_class$new(name, calc_func, breaks)
  ^~~~~~~~~~~~~~~
```


Hi all.  I'm following this issue with great interest b/c i've been a frequent user of the tufte ebook class. I just wanted to check my understanding here.  it sounds like the rticles package will be dropping the tufte ebook class, but that the new tufte package will be picking it up and that will be available in a day or two?  is all that correct?  (sorry for the question but i'm trying to get a staff member up and running with tufte ebook).  If it will be while before its available, is there a stop gap we can use with my new staffer (maybe using an archived version of rticles?). thank you.


Top :+1: merci :-)


Why is this a var and not just `function addResult...`?


Did you ever get this to fail again? I've tried a few more times, but I
haven't seen the error.

Eric

On Wed, Jan 6, 2016 at 12:53 AM, Eric Bishop <ericpaulbishop@gmail.com>
wrote:

> I'm not sure that it's anything wrong with your environment. I have a
> suspicion that it may be intermittent, and have something to do with the
> number of build threads running. I think the error is whether another
> package which is not explicitly listed as a dependency, but still selected
> is getting built first or not in the build process.
>
> Right now I'm trying another new build and setting the build threads
> explicitly to 1 to see what happens.
>
> On Wed, Jan 6, 2016 at 12:43 AM, Eric Bishop <ericpaulbishop@gmail.com>
> wrote:
>
>> I just finished a new build completely from scratch and it seemed to work
>> just fine... I'm not sure what is going on here.
>>
>> On Tue, Jan 5, 2016 at 11:10 PM, Michael <notifications@github.com>
>> wrote:
>>
>>> Just failed again with exact same error.
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/ericpaulbishop/gargoyle/commit/3209523b9e82fd0dd71468323855fac4d74dab37#commitcomment-15280438>
>>> .
>>>
>>
>>
>


Fix added in master: https://github.com/molinch/FFImageLoading/commit/6632636cf8649e0087bb1f3117adaac1c0a5669b


@jglick I'm sorry I just don't know what the 'right' thing to do here is, it causes issues in the browser, too. Ideas?


Early-out if `!ids`?


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 120](https://teamcity.labkey.org/viewLog.html?buildId=295312&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 42 Build time: 01:17:23
### Failed tests
```
org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

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

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

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

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

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

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

```


My mistake: I first thought that relative eg_values would cancel out for both kings,
but now I realise this has an impact when computing the score when taking into account the mg_values.

So it might be better to start with master values for tuning. Also I adjusted the ck's to be of increasing value.


paragraph added.


h


accessor
https://node-app-zenithtekla.c9users.io/players/23


And i know why xD. Its a source


Sure, `foo` is definitely better than `poo` :)


Double commas is invalid SQL syntax. :)


There is no other stable way. You might be able to check if the exception message is "canceled", but I'm not sure that will work 100% of the time.


A LOT xD lololol


Oh god.. love to have you back :)


Is it better to copy/clone the original input element and then change attributes on it?  Just in case the original html changes?


@deads2k, assigning `Resource` to `Kind` doesn't look right. How about adding a `Resource` field to `StatusDetails`?


First parameter should be `$mail`.


These two lines emit warning 55:
```
File "tfloat_record.ml", line 19, characters 8-53:
Warning 55: Inlining impossible in this context: Function information unavailable
File "tfloat_record.ml", line 20, characters 8-54:
Warning 55: Inlining impossible in this context: Function information unavailable
```
is that intended behaviour (and, if so, would it be OK to use `ADD_COMPFLAGS` to suppress the warning?)


Well what does it look like as a full tree? Maybe it looks better then.


You see the instanceof Living?
This is there because only living entities can activate it.


Can you hit `platform/{android,linux}/scripts/configure.sh` too?


Since you are effectively starting with an empty `stats.allClaimed` array each time this function is run, can you think of a more appropriate array method than the following use of `.forEach()`?


Is `setIfHostnameValid` expecting true in the event of an invalid hostname?


JSDoc for this method


escape не нужно


LGTM


`unsafe_unretained` 😱


Looks like you're missing a period at the end of the sentence in Line 4108.


:) yes, all the tests pass!


early-out if `!attrib.specified`?


Okay I blame ..


Done. Thanks.

2016-01-06 16:22 GMT+02:00 ingbrzy <notifications@github.com>:

> http://translators.xiaomi.eu/XML_MIUI7-Romanian-ro.html
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ygorigor/Romanian_MIUI7/commit/5ede43d06b94329b40d9e90a34a3343284f8794f#commitcomment-15291272>
> .
>


Fixed. :stuck_out_tongue_winking_eye:


All these double-brace tests should probably be single-brace tests.
https://serverfault.com/questions/52034/what-is-the-difference-between-double-and-single-square-brackets-in-bash


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


@nscottbishop I'm totally thrilled that there are frequent users :)  It will probably be more than a few days before `tufte` is released -- @yihui is saying that he'll put it online so people can start working on it today.

In the interim, I am happy to support it directly so you are able to keep using it without interruption.

The easiest way would be simply to not update `rticles`, and continue using the package as you currently have it installed.


Can use `progress.Updatef`


тут подключение пагинации не нужно. ее не будет на главной.


Yay for `.reduce()`.


Have we also considered auto-formatting the field onBlur? So we actually force the user to see the formatted input as well and later they might be less confused when they see it again?  Maybe just something to consider during acceptance testing.  Not a blocker by any means.


Fixed issues in your comment and the ugly whitespace. Thank you for catching the argument order!


Are there any plans up ahead for having an internal image viewer?


I think you need some training with git ;)


Да, спасибо за inspiration. Хороший код, тоже решил поизучать ES6 и запилить себе вкусняшку :)


The gc goroutine is never stopped either. I have to add something to close them both.


Revision returns the current revision of the KV the stream watches on.


Aren't these sin and cos Taylor series of the level 8 also valid for negative values (i.e. `-2*PI .. 2*PI`)?


Because `m.first_message` is None for the initial message, and as such all of that doesn't occur for the first message.

The very initial message sent to the subreddit's modmail *is* the first message, and as such that attribute is set to None. However if a message is a reply, *then* it is set properly (which is why what you mentioned doesn't need altercation).

I don't remember the logistics of it since I did this a while ago, but replies won't show if the initial message can't be shown, so to speak. Normally, the first message can be shown to non mods and as is is added to the query inserter. But without this change people that are mods but don't have mail permissions never get the initial message handled by the query inserter (I'm referring to `queries.new_message`)


Fix your URL ya scrub :stuck_out_tongue:


@alexcjohnson here there are:

![image](https://cloud.githubusercontent.com/assets/6675409/12153026/ab9f7af0-b485-11e5-81ee-241913dfb6dc.png)


Pressurestat (lgu1234) - x86_64-MacOS-10.9-clang: OK (1980 of 1980 tests passed)

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


I think it makes sense to limit these checks to entries where `SourceRepository` matches the repository we're pushing to; otherwise we brute-force `Stats` of every blobsum we know about before trying a cross-repository push.

I suggest changing `blobSumAlreadyExists` to take `metadata.BlobSum` as its argument, and doing this check inside `blobSumAlreadyExists`, to avoid the conversion to `[]digest.Digest`.


positionAt function of this class expects this.scale to be defined. Otherwise the feature tree will not work where this parameter is expected to be passed in "query feature". See line 40 of this file and feature_tree.js file `radius = (params.radius || 0) * (args.tileExtent || 4096) / args.scale;`


It's here I saw the RustLang framework doesn't have `(default)` in its description.


does jshint allow you to pass it files individually, or is running a background server (that I bet is watching all ./**.js files for changes) the preferred method?


what's the `|| true` doing?


Done in https://github.com/maxime1992/real-debrid-stream/commit/6fa3c0f7d95ffd1e746bc3f7c1343789de6ca5af


Klar, hab nicht darüber nachgedacht, dass die andere Figur dann automatisch vom Brett fällt.
Stimmt, dann kann man das einfach wieder rausnehmen


Full agree.
A split could make sense. I guess most application use either the mDNS announcing or the discovering.

I just had to hack the plugin to fit a requirement for a simple cordova based ios/android app.
Currently, I don't have plans to extend this PR. I hope somebody can jump in.


Sorry, I mean #870


@FGRibreau got it, good to know thanks


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10780](https://teamcity.mesosphere.io/viewLog.html?buildId=100618&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 212 Build time: 00:13:00
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


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


Hi @BenWiederhake, thanks for that advice.
I already fixed my problem, it is actually caused by this commit(https://github.com/vysheng/tg/commit/a9983e01c83b5befedff72c2e6220081d3bb76cb). It selected python3 as default while I wrote the code for python2.

Sorry for my silly mistake blaming this commit. Problem solved, so no need to open a ticket. Thank you.


private


Would better to have developers install bootstrap with npm or use a CDN than commit this whole thing to github.


Using the same `startingNumber` and `expected` for every test makes it very easy to cheat. Aside from cheating in a profession context it's easy to make a mistake when all your tests have the same expected result. I can mix these up and submit a patch if you are interested. Just let me know.

See search my solution for "cheat" http://exercism.io/submissions/6509763f2c664d38b34f87741163c563


We already have GoogleURL="http://www.google.com" defined in MiscTests.map, why we need to add it in this map?


showing how to make posts


That makes sense - but I didn't open the body before closing the head, I just put the JS snippet right there. The head was intertwined with the body before I was here.


Why was this reverted with this commit? Should this be like this or is this https://github.com/neobuddy89/shamu/commit/511a8f6b95cdf9dabcbf0784c8363a62e7960306 fixes it and yeah. Just wondering


about what?


I'm pretty sure I did this, but I think the ViewModel isn't supposed to expose any of the Model's properties to the View. This doesn't feel right... is it right??


Yeah, there are plans to implement it but I haven't got around to it yet. I will gladly accept a pull request, if you want to do it.


nice


I would merge this with SystemTablesModule into maybe GlobalSystemModule?


Ha, just realized I've been commenting under the wrong username


Yay! https://travis-ci.org/hpi-swa/smalltalkCI/builds/100495992


this should just be another field in GlobalSystemConnector


不错，很有深度的分析报告，两万五千个赞！


simple migration (task holder goes and back)


thanks i will. i used in a small xml.

2016-01-05 7:55 GMT+08:00 Brett Vickers <notifications@github.com>:

> Are you unable to use CreateDocument with a new root element?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hzmnet/etree/commit/0085e19da973489c507e77daa42f2ffb8084b6d7#commitcomment-15255316>
> .
>

--
Sincerely yours & Best regard!

*- Mr.Johan Hu -*
*Mobile:* 13510935999
*E-mail / MSN : * johan@ryron.net
*skype:* iRyron
____________________________________________________________
.
RYRON ELECTRONICS LIMITED
*Addr:*20#, East Block,GuangYe Bldg,Fuhua Rd.,Futian District, 518000
*Tel:*  (86) 0755-83978411
*Fax:*  (86) 0755-83974211
*Website* *:* www.ryron.net


change seem to allow config to pass and start working...i had just copied and pasted from another section.....prob a typo I haven't caught


Looks good and makes sense... just be mindful #11.


holy crap, you're absolutely right. I thought the green highlighted section included the code change from you.

sorry about that @jarantfos and thanks for pointing that out.


scheme能不能设成一个cordova参数输入？partner是数字，iOS 提交 App 时报错：URL schemes need to begin with an alphabetic character, ...


平顶山大学


That's a failure mode I have not seen before ... I am suspicious that the installations, which were not executed when the [build passed for me](https://travis-ci.org/dalehenrich/smalltalkCI/jobs/100487379), are causing the problem .... the GsDevKit_home code is installing things needed for both GemStone and Pharo (and presumably squeak) and they seemed to overlap when I scanned them, but that is one thing that is obviously different between the runs


Would it be useful to make it more generic in that case? I was thinking `ErrUnsupportedOperation "nats: Unsupported Operation for this instance"`;

Otherwise, just make it
`ErrSyncSubRequired "nats: Illegal call on an async Subscription"`?


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.88](https://pixelated.design/build/viewLog.html?buildId=308&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:22


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.85](https://pixelated.design/build/viewLog.html?buildId=309&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 269 Build time: 00:00:17


TeamCity Gensokyo Lobby :: Lobster Client [Build 0.0.1.32](https://pixelated.design/build/viewLog.html?buildId=310&buildTypeId=GensokyoLobby_LobsterClient&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


OK


I prefer the second option.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1174.4cbfb41da66c507ea5dd1e4a272dc80a00dd6a59](http://build.palaso.org/viewLog.html?buildId=67561&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout; tests failed: 2, passed: 813, ignored: 7 Build time: 00:35:24
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

GlyssenTests.dll: GlyssenTests.Rules.CharacterGroupGeneratorTestsWithLotsOfBooks.GenerateCharacterGroups_NumberOfNarratorsMatchAuthors_NarratorsGroupedByAuthor: <no details avaliable>

```


Swap Migration No Back


why was this done?  I really liked this.  Where have these been moved?


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


please remove the call to setFeature and getFeature in framework


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


haha ... yeah I found that the hard way :)


https://github.com/donejs/cli/blob/master/package.json

So what's the process of updating DoneJS?


I wonder if `SAMEORIGIN` would have done the trick


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


Hm, is it that there was a reference to extraClass() that was being ignored, and now no longer is?


Ok the build finished but BT share is wacking out.
I used this to build with:
https://github.com/WoundTight/zion959-proprietary_vendor_samsung

Your source with a few edits. Should I try using the trltetmo blobs?

Not sure if you use hangouts you can catch me at
dirtmodified57@gmail.com

On Tue, Jan 5, 2016 at 1:23 PM, ZION959 <notifications@github.com> wrote:

> thanks bro.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CMRemix/zion959-proprietary_vendor_samsung/commit/6f27109bd70df8f4c77b30c15786fcd256fa4d7b#commitcomment-15272655>
> .
>


Spacing?


@Connoropolous you might be interested to read this migration now... There were 200-300 odd mappings with no associated record, so I'm deleting them. I think it seems more drastic than it is. Anyways, this is YOUR baby :D


It's in https://github.com/donejs/cli at https://github.com/donejs/cli/blob/master/package.json#L52.


Sending a single encode version says "I expect the serialized object to be in this group and version, or get an error back".  Sending multiple encode versions says "apply the group version that matches this version".  This supports a scenario of explicitly serialize an object as a known group and version for which a conversion exists (since internal objects can exist in multiple groups and kinds).

For instance, an object Foo in group A and version v1 could have the same go representation in group B and version v2.  Saying "EncodeToStream(foo, unversioned.GroupVersion{Group: "B", Version: "v2"}) is unambiguous about what version it expects the group to be in.  typer.ObjectKind will return group A, but not group B.

`encodeVersion` is not required for codec versioning to be meaningful - a caller may not know what versions were provided to the serializer, but they know it can/should be able to convert the object to a particular group and version.


Will change


BackToSingleLib (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1465 of 1465 tests passed)

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


encodeVersion is a map.

On Tue, Jan 5, 2016 at 12:39 AM, Chao Xu <notifications@github.com> wrote:

> Why not targetGV=c.encodeVersion[0]?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/e98a2526d77800fcbf09a84531b11251b13a5bdf#commitcomment-15258425>
> .
>


Yes, will add some.

On Tue, Jan 5, 2016 at 12:35 AM, Chao Xu <notifications@github.com> wrote:

> Could you add some test for this Decode, like the TestDecode in
> json_test.go? I know there are tests in
> pkg/runtime/serializer/codec_test.go, but they are not directly test
> Decode with different defaultGVK and into.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/e98a2526d77800fcbf09a84531b11251b13a5bdf#commitcomment-15258390>
> .
>


Just want to let you know I managed to make this work with some help... Phew xD I was getting frustrated. Thank you for your patience


is it your server?


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


ObjexxFCL41 (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 85](https://teamcity.labkey.org/viewLog.html?buildId=295133&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 18 (6 new), passed: 275, muted: 1 Build time: 01:50:37
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <no details avaliable>

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

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

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

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

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

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


@icehess pleasse don't define `CUSTOM_SIP_VARS`. use the actual ones defined in route request


no i just tested it and it never came up till i entered 19132


Might want to spin out drawing the playfield into its own function to make render easier to parse.


also my server is OnyxTech still _indev_


I wonder if the `preload` flag makes any sense here. In order to include your domain in the browsers' HSTS preload lists, the `includeSubdomains` token must be specified as well.

Not even _pfsense.org_ is on the list at this time.

Source: https://hstspreload.appspot.com


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


`className` not `class`.  This is a super common mistake, I wish we could get it fixed.  @sebmarkbage


well..if you have already tested it...


ObjexxFCL41 (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-57.62%25-red.svg)


换JJ


@ZombiePower  then i know it's 19132 not 19133 lol


May be you should consider to use ReadAsync? Have a block sync read in async{} may block the threadpool thread, and cause performance issues.


Maybe we should consider to develop a ReadInt32Async()? See comment below.


But there's no family-studies in master yet..


We add that already in guiconfig.inc


函数名每个单词首字母大写


www


WTF?


Matched a previous, less clear name for the variable. Will update to `PACKER_KEYPASS`.


When are the BlockingCollection initialized?


I think we cant use the show and hide functions of flappy here . because in future it may be needed to change the styling from inline to inline-block. so i think we should not use those methods.


这是一个毕业设计项目....(基于安卓的疾病药品自查App)
 需求报告将在2016.1.6-2016-1.26号之前完成并上传


I'm trying to refer to the specificity of a particular version of code that will remain permanent over time. An example would be a GitHub URL that uses the commit hash instead of a branch name.


That's awesome news!


No, but the patient labels are there.


CBECC-Reintegration (kbenne) - i386-Windows-7-VisualStudio-12: OK (2336 of 2373 tests passed)

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


It may be helpful if you can add some comment on the thought behind the design of ServerNode/ClientNode.


oh you beat me to it...
damn it!
![screen shot 2016-01-05 at 9 42 44 pm](https://cloud.githubusercontent.com/assets/163977/12133648/49e12d12-b3f5-11e5-92e3-44425d224468.png)


yes.. that why i udated it in :)


lol okay :) @ZombiePower


I am not able get any idea to completely remove if statements. i only replaced one if with else


Currently, LogF is a sync interface. This may not be suited in async{} block.


I'll implement the case statement re-ordering. Which logic are you referring to, specifically, when you say
> From a readability standpoint, it would be better to move the logic for reading the file to a separate function.
I'll also probably implement the config structure regardless of moving things around, as it seems more organized.


Oh, yes.


Sounds good


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1126](http://teamcity.monogame.net/viewLog.html?buildId=13379&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


用法static+std::atomic<bool>来做


用原子变量的compare_exchange_strong来判断，否则会有race condition


The Latest and most up to date commit ....


还是放到外面，用static const std::vector<std::pair<std::string, uint16_t>> 来做吧


My two thoughts when writing it this way were:

- Since the encryption is not always applied, I wanted to be able to use the same variable, `writer`, later in the code without needing to know what layers are in it.
- I need to close each layer of the writer stack specifically, which is why I return a closer.

If you are in the office tomorrow I will stop by to discuss this in person.


This was a mistake I already found and fixed in a commit earlier today. An `openpgp wrong key` when trying to decrypt made it pretty clear something was wrong.


dds


"preventIndent" discussion here: https://github.com/wycats/handlebars.js/issues/858


Removed redundant method filterResults.


Aqui não mudei nada, são só espaços!


should be in static file


static const std::vector<std::pair<std::string, uint16_t> > kSchemeToPortMap = {{"http", 80}, {"https", 443}};


When a player eats another player's cell, they only gain 80% of the mass.


Not sure where dependencies are handled here, but this line requires the "next" branch of capstone


With both `detect_buffers` and `returns_buffers` on, redis returns a warning, because `return_buffers` overrides `detect_buffers` (https://github.com/NodeRedis/node_redis#options-is-an-object-with-the-following-possible-properties)


this will need to be changed, people might get confused if there are over 15 results :)


Based on current socket.io docs.


Necessary due to https://github.com/socketio/socket.io/commit/b73d9bea4efb48277eee685763026ff2df5a79ab


- Importante adaptar o evento de pó cósmico para imprimir o texto de output no lugar de output
- Fazer funções de atualizar unidades e inimigos


(did they always have the Duration right there in the title tooltip @aldenml ?
I see you also took advantage of that, that' what I had just coded there if you see the screnshot)


There's some weirdness with the event emitters and the lifting/lowering of so many servers in the tests; keeping these ones on separate ports from the other redis tests seems to avoid conflict.


lol glad u made it thru. :)


@dpb587-pivotal - Thank you, thank you, thank you.  This has bugged me for ages!


@Reedyseth I suggest using `trim()` on the content before checking if it's empty, so that if someone submits the template with a bunch of blank spaces the default value will still get set:

```php
! empty( trim( $_POST['options']['notification_content'] ) )
```


This doesn't do anything, does it? :P


Is this a intended feature or a debug print?


done


This is potentially erroneous and has been replaced with more efficient code


I've added a tag for the previous version of the package along with
instructions for installing it here:

https://github.com/rstudio/rticles/blob/master/README.md#sandbox-version

On Wed, Jan 6, 2016 at 2:52 PM, elbamos <notifications@github.com> wrote:

> @nscottbishop <https://github.com/nscottbishop> I'm totally thrilled that
> there are frequent users :) It will probably be more than a few days before
> tufte is released -- @yihui <https://github.com/yihui> is saying that
> he'll put it online so people can start working on it today.
>
> In the interim, I am happy to support it directly so you are able to keep
> using it without interruption.
>
> The easiest way would be simply to not update rticles, and continue using
> the package as you currently have it installed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rticles/commit/b5a41d054048ec60926165e881baeb89f318fed1#commitcomment-15298872>
> .
>


Most likely the tests should be adapted. @m-kuhn?


Is this import ever used?


This commit breaks `features/repository/show/hotspot_metric_results.feature:7` relies on the old HotspotMetricConfiguration creation flow.


That was included in the patch you sent us, @rafamanzo!


I don't think so... but I prefer finish the conversion to check if we need.


That's why it was not a commit ;)

Em qua, 6 de jan de 2016 às 18:02, Diego de Araújo Martinez Camarinha <
notifications@github.com> escreveu:

> That was included in the patch you sent us, @rafamanzo
> <https://github.com/rafamanzo>!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mezuro/prezento/commit/d94fed666c60bd97a5ce33bc7df7a7a46ed59eb1#commitcomment-15299046>
> .
>


2012-GlareSensorFix (rpg777) - i386-Windows-7-VisualStudio-12: OK (2337 of 2373 tests passed)

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


May not be erroneous but due to the type of project I don't like the look of just an else statement to reperesent an instruction.


You reversed the Sweeper direction here..


...but also reversed it here. Is there a reason for that?


@Golenko it was released a few hours ago under 1.5.6.


We can remove it then! We didn't realize it was there for debug purposes.


fix comment


Du hast das echt mit Angular-CLI durchgezogen? Respekt! :-)


Looks good!


Nice way to handle this.


TeamCity LabKey :: Sprint :: Premium :: Git sqlserver 2014 [Build 11](https://teamcity.labkey.org/viewLog.html?buildId=295335&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:06:34


could we leave the coach link up for now? "Attendees sign up is closed, but we are still looking for coaches"


Egads, so sad that we have to renumber manually.


@lexang16 exactly, simply still create a system variable or something else to store this product id.


They are not missing, they are simply not static inline in the header.


This line is *bad*, we should get rid of it.


Ut oh, this clobbered the SLFS update from yesterday!


Why this strange text output format? I'd make the function return a set of OIDs.


Would prefer doing this via SettingsProvider database, instead of a prop though.


I didn't really got the problem on the drupal org tread. Actually your version was working and the other not.
You may commit it to drupal.org. Maybe you could credit me there if the maintainer conflict get's solved somehow ;)


not all dependent objects, only relations.


Is this the correct include path? From my testing, the correct path was the document root, which contained the `wdn` directory, the templates_4.1 path was auto filled.


\o/


done


This is fine. My only thought would be to make it like our other applications where this language is shared (in case we ever have to update it). Doing it this way just means more work (but might also make it more reliable).


This is not the way to handle boehm.

First, Boehm has a low maximum for number of roots registered.
Second, Boehm won't scan the malloc'd segments.

The right approach is the following:

Only register the user scan function with sgen and use untyped gcj_malloc for boehm.


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


Includes must be always on the top.


What's the point of this change?


that doesn't worked


Oh... I made a big mistake


Nit: it's not obvious to me what `trait_ref` represents here -- the impl's trait reference? it'd be good to document it. (If it is indeed the impl's defining trait reference, you might also just look it up anew from the `tcx` rather than take it as argument.)


in that case it would probably be clearer to keep a counter of the number of _user docs you've seen, and then explicitly fail the test if `num_user_docs > 1`


Because later we also need to output function oids that are used in the
query, and with pg_dump to dump function DDLs. the client just parse the
json text to extract relation oids and function oids.

Thanks~
Haisheng Yuan

On Wed, Jan 6, 2016 at 12:10 PM, Heikki Linnakangas <
notifications@github.com> wrote:

> Why this strange text output format? I'd make the function return a set of
> OIDs.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hsyuan/gpdb/commit/e8e1a024a5af3423a0883129fef272c962f5ad29#commitcomment-15299602>
> .
>


Background:
InnoDB uses a `DELETE` and subsequent `INSERT` for `REPLACE INTO` which is very expensive for our table sizes and indices (30+ million rows) and results in a lot of I/O which our hoster isn't happy about since it can hit high IOWait rates. Having an actual `UPDATE` of existing unique rows avoids hitting the indices twice for each store operation.


FYI #4366 is ready for review


le point d'entrée de l'application est app.php et non bootstrap.php.
Je me souviens plus de la modif qu'on a faite aujourd'hui, je crois qu'il faut le faire dans le htaccess du répertoire app/


Yes, will change that.

Thanks~
Haisheng Yuan

On Wed, Jan 6, 2016 at 12:11 PM, Heikki Linnakangas <
notifications@github.com> wrote:

> not all dependent objects, only relations.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hsyuan/gpdb/commit/e8e1a024a5af3423a0883129fef272c962f5ad29#commitcomment-15299936>
> .
>


I'd go and rename this to MONO_HANDLE_ARENA_POP_RETURN_UNSAFE and just  MONO_HANDLE_ARENA_POP_RETURN.

To make it clear the the first has unsafe  behavior while the second not.


package.path = package.path .. ';.luarocks/share/lua/5.2/?.lua'    ..';.luarocks/share/lua/5.2/?/init.lua'


Why sizeof( float ) ?


As in your other PR, I feel uncomfortable that this `trait_ref` is escaping the `probe` where it was created. If it contains inference variables, in particular, that can be very problematic (leading to ICEs or other problems). It would be good to "sanitize" it somehow, ideally within the `probe`.


> Shouldn't this be r4 instead of r2 ?

@vjr, Vishal, yes, it shour. Would you like to offer a pull request?


Because we don't use long long and double in commands. Though 16 may be nicer.


Not quite. They are generated Remez's minimax approximations and are more precise: http://lolengine.net/blog/2011/12/21/better-function-approximations than Taylor.
Specifically these values are generated for range 0 ... 2*PI since this library has no need for the extended range.


Hey Daniel, you can contact me at lxbchrd@gmail.com ... but I should be ordering my own modules within a week... not sure about the pi-zero though because I still haven't received an update from the distributor about when they will become available.


It would actually be an error.... could be an assert() - but we don't do asserts except in the initialization code.


Nessun problema!😉
Il 06 gen 2016 20:27, "Matteo Di Pirro" <notifications@github.com> ha
scritto:

> Mi ero lasciato fuorviare dalla stesura in italiano. Correggerò nel
> prossimo commit.
> Il 06/gen/2016 19:43, "Davide Polonio" <notifications@github.com> ha
> scritto:
>
> > I nomi dei file non erano in inglese?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/BugBusterSWE/documentation/commit/48a50ec8fbdf257c20a190c0a944d68ec8568b23#commitcomment-15297520
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BugBusterSWE/documentation/commit/48a50ec8fbdf257c20a190c0a944d68ec8568b23#commitcomment-15298358>
> .
>


GroundDomain-Output-Variable-Name-Changes (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1996 of 2001 tests passed)

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


Aligning to integer size generally makes more sense. But nvm.


This commit should not have been applied to `devel`


Works for me.


Looks good; thanks!


This commit should not have been applied to devel


you rock mang


this commit should not have been applied to devel


This change is wrong for two reasons.

First,it's duplicating the allocation code, that's a no no.
Second, it handles OOM incorrectly.


Output looks like this:

```
$ grunt ghostinspector:test2
Running "ghostinspector:test2" (ghostinspector) task
Executing tests...
>> Test "Google" (53cf58fc350c6c41029a11bf) passed
>> - Screenshot comparison failed
>> Test "Yahoo" (53cf59e0350c6c41029a11c0) passed
>> - Screenshot comparison failed

Done, without errors.
```


This hunk was already in, did you `git pull -r` before working?


Same problem as the previous commit, tons of duplications and bad OOM handling.


Output looks like:

```
$ grunt ghostinspector
Running "ghostinspector:test1" (ghostinspector) task
Executing suites...
Suite (53cf58c0350c6c41029a11be)
>> Test "Google" (53cf58fc350c6c41029a11bf) passed
>> - Screenshot comparison failed
>> Test "Yahoo" (53cf59e0350c6c41029a11c0) passed
>> - Screenshot comparison failed
```


I thought maybe Retrofit can send a special Throwable like `NetworkSomethingException`.


Jordan and I discussed this yesterday offline. Later in the evening I came up with a better design that resolves our disagreements and gives an even better design!

Michael

> On Jan 5, 2016, at 10:30 AM, Jordan Rose <notifications@github.com> wrote:
>
> I really don't like the implementation here—forcing the LargeIndex on the integer case just seems wrong. I feel like you should hardcode the 4096 limit based on the actual page size of the current OS, then use PointerUnion::getAsVoidPointer for both the pointer and the large int cases.
>
> Alternately, just use PointerUnion<PtrType *, Fixnum<31>>, which in practice will be good enough.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/9fa6e31f9ec42e010cf1577a1b44e2cabfd1cbfc#commitcomment-15271573>.
>


I'm not sure if this function would get called when evaluating the results of a continuous changes feed, but there are valid scenarios where you'd see more than one _user entry in that scenario.


In general, we are trying to move away from these wrappers, except when running under the debugger, which depends on them to do async stack walks.


this is the real culprit.... I'll delete the other comments. This branch should not have been merged into `devel`, since it's just bumping version number, but `devel` is already slated to be a 2.x.x release. I'll work on reverting it...


This comment does not apply, just remove it:

```
-    # disable-policy-batching ipv4
```


I did `git pull` and thought I was up to date.  Neglected the `-r` option; apologies.


Test is failing:
`Error: expected '6' to equal 6`
Are filter number arguments converted to strings?
Should just expect '6' [like it's done here](https://github.com/lazopm/nunjucks/blob/0f9f94887ddbd5203614fe48dfe39eeca76cc68d/tests/filters.js#L26) ?


it starts in `reg` and depends on recv_len for how many registers it reads (the word size is 8 bits).


These are deliberately different from alphabetical order; it proves that next/prev_item use the pagination order, not the "children" order.


They haven't switched to their new site yet, have they?


leftover actually... I was turning this table into constexpr and for that I need to initialize all members.


Actually I think all these status checks are somewhat all the same. I would like to see them extracted in some kind of arrays, e.g. "editButtonEnabledStatus" or even better into a "Project" object which wraps the project json returned by user and aggregates all the status checks into on single "class".


This commit fix #658.


Oh. It appears that the code was buggy. If we register an opcode emulation with no_throw enabled, we call directly to the native func instead of the wrapper. The previous code was inlining it either way, resulting in bad performance. For these fast opcodes, since most of the time is spent in the wrappers, inlining improves performance by 50% (assuming we have the same behavior when it comes to the LMF). I still think that for these fast opcodes we should implement them natively instead (which would remove this wrapper problem altogether and also be faster), while for the others, where inlining only adds about 10% performance improvement, we should go for the reduced code size and don't inline.


Another option that seems pretty perfect:  http://fortawesome.github.io/Font-Awesome/icon/history/


...and user is admin


Yes! I can see that what you removed/added.

RG reading  the specifications of the spectrometer from a xyz file: this sample code is old one and new "AutoIntTime.php" module read the values form the the .txt file.

Just one point: you did, your commit (i.e., changes) on a new branch (please see forks up right corner of this page)

i only made a simple change on this branch that you made (i added a comment line at the end of the file) to show you how you can track my update/work

let's keep going our example conversation here! but please do not forget that our accounts are public


Maybe one more test "should not display the publish-defer-button when the user is no admin"


I suspect we do not want to create two inference contexts here in any case, that seems like a big recipe for trouble.


speling error :)


In the future, since this is a trivially small branch and change, I would recommend using `git commit --amend` combined with `git push --force origin bump-version-number` to fix typos and trivial changes like this. If you're on a branch created to do something like bump the version number, then no one else will be likely to also be working on that branch, thus amending commits (or interactively rebasing them to edit, fixup, squash etc. commits) followed by a forced push is unlikely to cause confusion or difficulty for others.


I like! :-)


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


GroundDomain-Output-Variable-Name-Changes (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1465 of 1465 tests passed)

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


That's rule was wrong. Remove last slash from regexp pattern.


lgtm


Ok, I walked through all the textures. And to be honest, I don't see any good reason why the particular reason why those particular liveries and not the others should be left in:

 - There are liveries with different resolutions, but the restricted set does not consistently prefer the higher resolution ones.
 - There are “clean” liveries and “realistic” liveries with some wear and tear overlay added. The R one was preferred where two variants existed, but the set contains some “clean” liveries too.
 - Some liveries add the “floats.png”, possibly indicating they were intended for the float version, but it does not seem to make any difference anyway; not all of the liveries left in use it.
 - I don't see any difference in quality that the choice could reflect.

Other observations I made while checking the liveries:

 - The liveries that show custom callsign (Baroudeur, Blue2, BRT Virtual Regional and Trans North) only show the callsign on underside of the wings, but nowhere on side of fuselage or vertical stabilizer. That might deserve fixing.
 - Texture for stopped propeller is included, but texture for spinning propeller is not, so the rotating propeller does not match the look of the stopped one in those liveries that have different markings than the typical two stripes near the end—there are several with large black and white stripes over the whole blade.
 - Three liveries (Air Tahiti - F-OCFJ, Air Tahiti Nui - F-OIQF and Air Moorea - F-OIQP) have completely white propeller blades, which looks like a mistake (I doubt anybody would remove the stripes; they are a safety feature to make the propeller more conspicuous when it spins).
 - One livery has non-ascii name (Armée de l'Air, F-RACD.xml), which is encoded in utf-8 (and the xml properly marked), but FlightGear (and I updated it from next) still treats it as iso-8859-1, so it renders wrong. It does not seem to break anything though.
 - One livery (Wideroe), which was not removed, shows strange lighter speckles on the underside of wing and lower fuselage. I don't see them in the bitmap, but there might be some part purpose of which I didn't understand.

Since the selection appears arbitrary, I would definitely be for restoring the complete set. We could possibly leave the duplicates out, but the ‘R’ variants don't look all that nice (the resolution is only barely enough for this), so I'd probably leave them in anyway. It's just a few liveries anyway. And then it might be nice to add the custom callsign on the empenage (it has to be on the side or tail according to the regulations).


@borshop: retry


@thebigsmileXD
All Fixed
Check again plz


> @deads2k, assigning Resource to Kind doesn't look right. How about adding a Resource field to StatusDetails?

I agree, but I didn't want to change existing behavior a the time.  Especially since its an unversioned type.


add-default-build-type (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1465 of 1465 tests passed)

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


Yes, it can be, but is rare, I have tested with Chromium version 49.0.2615.0 and I'm not having the same problem, you can try to download it again, or directly download the [ZIP file](https://github.com/bumxu/MixcloudHarvester/archive/master.zip) (Chrome folder) with the new changes made by [Waterflames](https://github.com/Waterflames).

Regards.


As I said in 17ede96, this commit and the three before it would have benefited by being squashed or simply amended. Now there is additional noise and a few invalid states in the project history. Not a big deal, but as the project grows and we take on more collaborators, these sorts of best practices are worth considering.


Oy, this was resolved in a later commit but I failed to edit the history.


Put this inside "{" so its readable.


so up to this commit mostly makes sense, though I did not thoroughly study the `specializes` function yet


you can see how the project tree is growing here: https://github.com/hamnan1945/test/network

we may want to work on one branch (ex: default branch called "master") and go till fulfillment.


Add a new line and put this inside "{" - avoid these long lines.


PocketMine-MP?


Does the like() mean that it will match any content type containing the REGISTRY_CONTENT_TYPE string or will it have to match exactly? We would want it to match any content type containing the string I would think. Does this support regex like "registry.*"?


Think this should be just "registry" since we want to filter out all registry entries not just the federation ones of "registry.service"


:yen:


..and periods.


Hi, I've been using jupyterhub for some times now, and I've just updated jupyterhub on my development server and this update create some problem for me (it probably means I don't understand something).

If the /login is always handled by the LoginHandler how can we use a homemade authentificator/handler? This handler will always capture the request before my homemade one, right?

Thanks for the feedback, I really need it...


saw approval from andytill
at https://github.com/basho/riak_kv/commit/777212a796c3038c08ba3c2e0b0c668a1ddc7c97


merging basho/riak_kv/fix/az/refactor-dialyzer-coverup = 777212a7 into borshop-integration-1317-fix/az/refactor-dialyzer-coverup


basho/riak_kv/fix/az/refactor-dialyzer-coverup = 777212a7 merged ok, testing candidate = 5a531e6b


Just for the record, I did some tests to scratch my itch :-)
`new BigInteger("2").pow(32).longValue() - 1` is about 3x slower than
`BigInteger.valueOf(2).pow(32).longValue() - 1`, which is about 2x times slower than
`BigInteger.ONE.shiftLeft(32).longValue() - 1`

And `0xFFFFFFFF` and `0xFFFFFFFFL` are indeed different.


My concern is that this will break existing callers passing in the PhoneNumber arg


Ah, yes, I meant "vector" :-).


>Since the selection appears arbitrary, I would definitely be for restoring the complete set. We could possibly leave the duplicates out, but the ‘R’ variants don't look all that nice (the resolution is only barely enough for this), so I'd probably leave them in anyway. It's just a few liveries anyway. And then it might be nice to add the custom callsign on the empenage (it has to be on the side or tail according to the regulations).

Nice analysis @jan-hudec
I definitely agree with leaving "judgment" of aesthetics out of here, and base on purely technical, the liveries are good and work.
Certainly having a GPL livery, that can be improved, is a motivation for someone to do a better job at it. But in the mean time is there, and its properly installed and tested to work.

I would send later today a commit restoring them in master and a PR to bolan's

Thanks again
@IAHM-COL


Grammar nitpicks:  s/and hyphens/hypens/  and "Please try another name" needs a period at the end.


sorry, they use the Min version which is the older version on the FTP(before my changes). I can make it a separate case and re-minify if if you want


Should this be patter.startsWith(....)?


And does this still work if you don't pass in AssignedDID or StatusPostBackURL?


Jo, schaut mal hier: https://github.com/Angular2Buch/angular2-forms/blob/master/ember-cli-build.js#L12
Da steht, wie man weitere Frameworks in das `vendor/`Verzeichnis packen kann.

@d-koppenhagen @fmalcher @JohannesHoppe


This patch isn't necessary any more.


It's not always Retrofit that cancels as it propagates downward into OkHttp's `Call.cancel()` and this has been brought up before for OkHttp but it too can't always control the exception that is thrown. Retrofit might be in the middle of reading the response body and have just a normal IOException make its way up.


Thanks!  I just tried it and it works nicely.

I just read about this, which explains why setting LC_COLLATE (which you do through LC_ALL) manually is necessary:
https://docs.python.org/3.5/library/locale.html#background-details-hints-tips-and-caveats

Should we move the call to `locale.setlocale(locale.LC_ALL, "")` at the beginning of the program, so that we use the user's settings (e.g. for times and dates) everywhere?


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


I changed back to the old way. If you got time, check it here to make sure there are no errors: https://app.roll20.net/join/1166262/hRMq5g


as it stands, this one works:
http://www.nueratrailerparts.com/c-3026-custom-trailer-hitches.aspx

and the code thats on github works for their new site.


Thanks! I just tried it and it works nicely.

I just read about this, which explains why setting LC_COLLATE (which you do through LC_ALL) manually is necessary:
https://docs.python.org/3.5/library/locale.html#background-details-hints-tips-and-caveats

Should we move the call to locale.setlocale(locale.LC_ALL, "") at the beginning of the program, so that we use the user's settings (e.g. for times and dates) everywhere?


@thebigsmileXD
This Method of Redstone Calculation can have both logical and performance
Think it out for many days


needs indentation


That would have been a much better model name.


Yes, I believe this is correct. The result is `www`.


:blush: :bow:

Also, holy :cow: -- where do you find the energy to build all these things, @IonicaBizau?


Could you add some tests for lax mode checking?  e.g. string length, invalid chars


Yeah that was my main concern, that we broke their live site by supporting their dev site. Please make it a separate case statement and I will communicate the changes to them.


Initial Project using Login Template in Android Studio.
Minimum SDK: API 15 - Android 4.03 (IcecreamSandwich)


Whoops, lol


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


Nice to see you back!


@aodzip @keithkfng Why can't i cancel builds in jenkins?


testing writing a note


It's hard to notice (especially on your monitor), but the sub nav bar doesn't *actually* go away when the subheading and search bar are gone (the background color of the sub nav is still light grey).  Other than that, this looks great!


I cancel it


Don't you know how to use Jenkins?! You simply click cancel while the build is running. Cancel means the little X button next to the progress bar by the way.


And add more slots xD


@Akujin I agree, but I don't want to keep piling features into ActiveRecord, and there are a few other alternative ways to do history (e.g. ActivityRecord writes to a central stream of changes rather than storing complete old versions).

There is a [next-generation implementation of ActiveRecord](https://github.com/JarvusInnovations/emergence-skeleton-v2/tree/master/php-classes/Emergence/ActiveRecord) underway that implements a new stackedConfig called `$behaviors` that lets you plug external classes into an ActiveRecord's lifecycle. Traits don't seem to be a good solution since they just add members to a class without any way for them to self-hook into the classes lifecycles, unless we use a ton of reflection which seems expensive performance-wise.

For legacy reasons though this version of ActiveRecord/VersionedRecord need to stay how they are and this commit represents an incremental but compatible improvement. The next-generation version above though is where we can work out better ways (and I think behaviors feels good so far)

I don't agree with your example though of having a Class C extend Class B with Class B being versioned and Class C not being versioned. If Class C shouldn't have all the properties of Class B it shouldn't extend Class B. Extending Class B is a contract that Class C is everything Class B is and more.


I think he doesn't have Administrator Prev


There is no button for me, thats the thing


Strange thinking of the func return value as a parameter but it makes sense to have consistent handling of them. :+1:

May be something like 'bindings' could be another name to use but perhaps a bit confusing too.


Cool. Are you going to start working on 0.3.x version?


9999 is still needed, you cannot use 10000 - you need to have a upper bound for the maximum part size for 5TB which is 5TiB divided by 10000 parts.

9999 is needed for anything lower than 5TiB since all values are not divisible by 10000 so you always end up having an additional part. To cater for that you have to have 9999 and then the last part is always smaller than the rest of the parts.


I think we can remove all thes delegate tests that are just checking for false


@phausler Howdy!  I know this is low on the totem pole with everything else going on, but any additional thought on my comments?  Thanks!


Damn probably my fault then


Good catch. I removed the ReadWrite scope.


LOL


whaou


This is still not fixed. We should still update the /etc/hosts file with first IP for backwards compatibility


Pressurestat (lgu1234) - x86_64-MacOS-10.9-clang: OK (1980 of 1980 tests passed)

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


we also can notify each other by  @

@NocusNibbes


Yee, that was really stupid from me, sorry about that.


Sven-Eric;
Thanks for this. As normal, you're reading my mind. We already do the quality conversion as part of bgzip and indexing so everything downstream can assume sanger:

https://github.com/chapmanb/bcbio-nextgen/blob/master/bcbio/ngsalign/alignprep.py#L434
https://github.com/chapmanb/bcbio-nextgen/blob/master/bcbio/ngsalign/alignprep.py#L87

The goal here is to try and make the preparation fast enough that we can always do it, which will enable more parallelization and hopefully better runs as well. I was hoping SDF indices from RTG would be a good solution. They are faster to generate but take ~3x more space than bgzipped fastqs. @lenbok, are there ways to compress RTG fastq file storage to make them more disk friendly? I love the built in read indexing and retrieval.

We do have auto-detection of format, although only as a check instead of automatically setting it:

https://github.com/chapmanb/bcbio-nextgen/blob/master/bcbio/ngsalign/alignprep.py#L87

Maybe we could add a `quality_format: auto` option to guess and set quality formats if folks want to trust bcbio to figure it out. Would that be helpful and save upstream processing?


I used the musl fix to successfully build libstd with a musl toolchain for an ARMv5 target (https://github.com/rust-lang/rust/issues/30710#issuecomment-169210254). Thanks!


I guess the bugsnag report was broken then...


Yeah. The CMS pull didn't suit me because it comes with too much DOM baggage. I just want a `<p>` :crocodile:


The naming in database data dictionaries is similar so I guess this came up from there. Also its similar in [JDBC docs](https://docs.oracle.com/javase/7/docs/api/java/sql/CallableStatement.html).


Thanks for the quick fix :)


Why both Retrofit and OkHttp can't throw `new NetworkException()` instead of `new IOException("Canceled")`?


the pending designation was leaving lots of stack traces making it hard to debug my relevant tests.
i've gone through now and marked those tests with 'xit' or 'xdescribe'  which still reports them as pending but doesn't attempt to execute, so provides a much cleaner output.


Done.


Logged at https://github.com/silverstripe/silverstripe-framework/issues/4895


I don't like this empty error. We could have something like `Some of the hosts couldn't be removed`. Or we could collect the hosts in error and return a single error with all hosts in error.


`continue` if `!this.isRelevant_(...)`?


pie


We added "claim the PR when you want to review it" for our review process in today's storytime.


hahaha, lol, not working on PocketMine anymore!


Cool, looks good, thanks.


I ONLY look at these like a full tree. Testworlds...
On Wed, Jan 6, 2016 at 2:48 PM Zsashas <notifications@github.com> wrote:

> Well what does it look like as a full tree? Maybe it looks better then.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/L0neKitsune/Botanical-Addons/commit/c430dea7976317115a0dbf20d230d6e0acb4a133#commitcomment-15298809>
> .
>


Excellent. So it fails on the line:
```
mpz_pow_ui(expo, a_final.get_mpz_t(), ((mpz_class)((p.as_mpz()-1)/2))
```
which is here: https://github.com/symengine/symengine/blob/8b218d053b34321ddb3584fb0d3cb34e2743a53e/symengine/ntheory.cpp#L1539 and you are using a C cast there:
```
(mpz_class)((p.as_mpz()-1)/2)
```
You should never use a C cast in C++ code. In particular, you should not use any cast here at all. You are probably casting a C++ expression template (which is not the `mpz_class` type) into `mpz_class` and that's why it segfaults. Just remove the cast and things should start working.


Mein Held! :smile:


add-default-build-type (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2033 of 2033 tests passed)

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


Added one line to the Readme.


wow! cool!


Nicely done...


@legoboyvdlp You overdeleted :P

     osgDB ac3d reader: could not find texture "10_spaces_dual.png"
     osgDB ac3d reader: could not find texture "hatchback_red.png"
    osgDB ac3d reader: could not find texture "hatchback_red.png"
    osgDB ac3d reader: could not find texture "hummerh2.png"
    osgDB ac3d reader: could not find texture "van_blue_dirty.png"

I'll restore these in a minute


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.444.0](http://build01:8888/viewLog.html?buildId=13768&buildTypeId=ConfigInjector_CI) is now running


Ok, I found it in the last change in this series :smile:

It would still be feasible to keep track of this at the call site, but as I said it's not a blocker; happy for this to go in as-is, and if I really feel strongly about it I can make the change myself.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1175.df53bb29a17efc0a2fc6ad4d8e1e97831e5fb85c](http://build.palaso.org/viewLog.html?buildId=67564&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 811, ignored: 7 Build time: 00:17:44


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.444.0](http://build01:8888/viewLog.html?buildId=13768&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:30


-> PlayersStats.empty


Needs to also be changed in Makefile (`CCFLAGS+=-DCOLOR_OK` -> `CCFLAGS+=-DCTEST_COLOR_OK`). Or maybe even removed from there as we add it in main.c?


add-default-build-type (nealkruis) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


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


Should the above 3 lines be deleted?


Not there XD
I'll change it to Notes.


This looks amazing :sparkles:


Line had extra characters that shouldn't be there.


`Separator` instead of `SEPARATOR`


Ah, okay. My bad, you are right. :)


You want to use `add_filter( 'template_include', '__return_false' );` here. I could not find a way to stop the functions.php being loaded. The best solution maybe to add a check in the theme.


My Babel does not, and I'm somewhat confused why it would say that -- method names and local variables are decidedly not in the same scope.


If only $input.val() is only ever used here then makes more sense to pass in the val instead of $input?


Picky picky (it bothers me too, I wonder if I can create a twig template for that node to remove the baggage).


For ```.read``` it is pretty much the same thing.


fixed
https://github.com/mherweg/RJBB-fg-CustomScenery/commit/899d461641d548bfafd8ead6cd1fc5a481eb9286


Ahahah great job man!


just an update, I still haven’t been able to get this working in osx. I
pulled a copy of this PR to a remote cluster that’s running CentOS on
linux. This doesn't just quite as soon as it starts like it did on OSX. It
seems to get through all of the documentation (at least it gets to 100%
with a bunch of warnings but no errors). However, just this is how it
finishes the job:

[image: Inline image 1]

Not sure if that's just a driver error on linux (it's using network x, so
there are graphics available but they can be wonky sometimes), or something
wrong with the code.

Is there anything I can check to make sure that it worked properly or not?

Chris

On Sun, Jan 3, 2016 at 9:10 PM, Eric Larson <notifications@github.com>
wrote:

our code should only call from matplotlib.pyplot, not matplotlib.pylab
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mne-tools/mne-python/commit/4b866e009bbbaa3192906e4655280407f147deb9#commitcomment-15236351>
> .
>


Hi there, I'm a Java dev and also have a Waterrower. I was looking at this project and wonder if you would mind me in future forking it so I can maybe adding some extra features. It also gives me chance to pick back up my old Android skills which haven't been used for a 3 or 4 years. I'm not sure when I will get round to it but it is certainly something I may be interested in looking in to as the S4 is basic to say the least! Steve


Hi @zbeekman this is very interesting. Can you elaborate more? I have not completely understand the *desiderated* result.


Oh ok. I can only see the individual textures lol.


Much Thank !


What location are you describing?


You were supposed to add a file describing your hometown. What town are you describing? Try to follow instructions.


Fixed the spelling of Atlassian.


size also verify if its greater than the maximumPartSize.


If they don't attempt to execute, there's no indicator when they start passing again because they aren't being run. The way they were marked before, if you implemented a pending spec and it worked, it would flag it as a test failure so that you could either remove the pending designation or figure out why it's returning a false positive.

You can avoid the stack trace spam by running subsets of specs via tags, or running specific specs by line number, but I'd prefer them to still attempt execution so that we don't lose the ability to react to unexpected changes in their status when running the suite.


I believe that this revert will have to be reverted right before we finally merge `devel` into `master` for the first 2.x release. It's really confusing, but see: https://git-scm.com/blog/2010/03/02/undoing-merges.html and if you really want to hurt your head, read this: http://opensource.apple.com/source/Git/Git-26/src/git-htmldocs/howto/revert-a-faulty-merge.txt


missing space after '>='


@Jetski5822, @sebastienros

That's what i've seen yesterday. Seems to be due to the `IAssemblyLoaderContainer.AddLoader()`. See the dispose action here https://github.com/aspnet/dnx/blob/dev/src/Microsoft.Dnx.Host/LoaderContainer.cs, where scopes need to be disposed in the right order.

So, you can have issues if used in concurent tasks (as when activating shells), especially since you have more features...

So, just for infos, seems to work if you only add a lock in `ExtensionManager.cs` in `BuildEntry()` when for each loader we use `loader.Load(descriptor);`

Best


aggregatedSize is a better name just 'size'.


Just, you know, make sure you have tests. :)


Commit d8dbd8e should have the correct spelling. Fixed the spelling in the title of the pull request too.


The symbol is actually called REAL_TYPE rather than "DREAL_TYPE" (copy&paste from "-DREAL_TYPE").


```
// aggregator is a simple through stream that aggregates
// chunks of minimumPartSize adding up to multipartSize
```


https://github.com/AndreyShpilevoy/DemProject/issues/32
https://github.com/AndreyShpilevoy/DemProject/issues/28


Ah the logic has changed a bit in this one you are keeping upper bound as is. You are instead dividing the upper bound as well with minimum part size to know the smaller chunk possible i.e fixed to minimumPartSize.  Which is aggregated later internally before uploading. This is correct my mistake.

But why not replace the previous function of calculatePartSize with the present one and avoid declaring at multiple places.  Another thing is you should elaborate the purpose of calculatePartSize - writing one a single line confuses what is it uses for and why are we doing this approach.


fyi @mdtusz


can the chunk.length be smaller for the last part? should it always be exactly minimumPartSize?.


shouldn't the updated version be used?
https://github.com/tpoechtrager/cctools-port


Added smartks fixed some bugs :wub:


add-default-build-type (nealkruis) - i386-Windows-7-VisualStudio-12: OK (2001 of 2001 tests passed)

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


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


Remove the new line.


omg, you got the tests to run. :+1:


There is a mistake, mousePosition.x should be mousePosition.y in "y" value calculation (line 811).


Also, thanks for fixing 6 year old hard tabs. :)


Ok, well I am not sure I quite understand what action we should or should not take then, @paddybyers are you happy for this to be left like this for now, or do you suggest an alternative?


la modif qu'on a faite aujourd'hui c'est pour un autre problème. C'est un problème de configuration qui change entre apache 2.2 et 2.4.

mais il faut sans doute aussi changer le app/api/.htaccess mais autrement que comme ça pour apache 2.4

je vais chercher comment annuler la pull request


Where would be a better place for this view logic?


> Can this be pulled into the 2.x branch as well?

@superkhau Can you either do it or show me how to?


agreed, i just documented it like this because i wasn't sure if you wanted to combine the modelsim and questa into a single "mentor" makefile since they are identical now, or continue to daisy chain them


Doh! Thanks @mvgrimes. removing the old logic and merging.


This doesn't need the `getPointer()`.


TKS :+1:


Why the new `createCart` function?  You only seem to use it here, so:

    var cart = new Cart(order);

should be sufficient.


IKR? And always, only rails placed between 2 change their direction.


Actually i also work on PM alot last time.
i'll fix it.
Argument 3 passed to pocketmine\event\entity\EntityLaunchFishingRodEvent::__construct() must be an instance of pocketmine\entity\Projectile, null given


It really does.. Tested so long, the bugs are MINI


hihi ;)


All thanks to @aodzip


And of course you guys for testing bugs and fixing mini bugs


`master` and `devel` are protected branches. All work gets done in topic branches and applied to one or both of `master` and `devel`. If you have a small topic branch in an organization, you can safely re-write history if you are confident that no one else is using that topic branch. (If you have a private fork, then you can **guarantee** no one else is using that branch.) When you make small mistakes like this, it's cleaner to erase them from the project history by amending or squashing commits. However, this will get you into trouble if you force a push to a repository that others are actively working on.


Hi Steve,

Coxswain currently only does what I need for my workouts. Suggestions or contributions are always welcomed.

Have fun coding on your fork!
Sven


Handling a `.catch` here is a nice-to-have, IMO.  Might be worth checking the failure path here in acceptance testing.


between expressions like if/for/foreach and the opening bracket should be a space


And me, i did some toggles too :P :tada:


Ah what the hell! I thought I fixed that already. Dammit, haha. Thanks!


I am - its mostly unchanged from what is currently there. I am checking that I can update my libraries that use Scala.Rx then Ill create/accept the PR.


XD


Yes
thebigsmileXD help the Redstone much.

PS . I was a mini bug fixer before ahahaha


Thanks. Fixed.


Commit message could be better here:

This validation is introduced to ensure users cannot vote for quotes more than once.


Tests required


Hah, I changed it in this commit, undoing.


Want more screens?
![minecraft_ windows 10 edition beta 06 01 2016 21_55_31](https://cloud.githubusercontent.com/assets/8733998/12155302/b14f72e0-b4c4-11e5-961b-3f098f38ce0c.png)

As you may see here, the torch at the bottom of the image doesn't invert. But it should
![minecraft_ windows 10 edition beta 06 01 2016 21_55_37](https://cloud.githubusercontent.com/assets/8733998/12155303/b1543104-b4c4-11e5-865a-d54a1e65ff77.png)
![minecraft_ windows 10 edition beta 06 01 2016 21_56_01](https://cloud.githubusercontent.com/assets/8733998/12155304/b15535ea-b4c4-11e5-94f5-1422563d17f1.png)
![minecraft_ windows 10 edition beta 06 01 2016 21_56_02](https://cloud.githubusercontent.com/assets/8733998/12155306/b1597998-b4c4-11e5-8fb0-d19927e9aef6.png)
![minecraft_ windows 10 edition beta 06 01 2016 21_56_03](https://cloud.githubusercontent.com/assets/8733998/12155305/b156bad2-b4c4-11e5-9e5a-737bcdf26e6f.png)
![minecraft_ windows 10 edition beta 06 01 2016 21_56_04](https://cloud.githubusercontent.com/assets/8733998/12155307/b15a70b4-b4c4-11e5-88b6-bb0f0912449c.png)
![minecraft_ windows 10 edition beta 06 01 2016 21_56_06](https://cloud.githubusercontent.com/assets/8733998/12155310/b17c2d3a-b4c4-11e5-907a-c62295ac07a5.png)
![minecraft_ windows 10 edition beta 06 01 2016 21_58_28](https://cloud.githubusercontent.com/assets/8733998/12155311/b1860878-b4c4-11e5-84cd-2b495786a0a3.png)
![minecraft_ windows 10 edition beta 06 01 2016 21_58_48](https://cloud.githubusercontent.com/assets/8733998/12155308/b17779a2-b4c4-11e5-9cd1-f47041b633d9.png)
![minecraft_ windows 10 edition beta 06 01 2016 21_59_38](https://cloud.githubusercontent.com/assets/8733998/12155309/b17ae38a-b4c4-11e5-800a-27b28b72ad7d.png)
![minecraft_ windows 10 edition beta 06 01 2016 22_00_56](https://cloud.githubusercontent.com/assets/8733998/12155313/b18e47f4-b4c4-11e5-9f9a-8682805c9a39.png)
![minecraft_ windows 10 edition beta 06 01 2016 22_01_49](https://cloud.githubusercontent.com/assets/8733998/12155312/b18dc694-b4c4-11e5-89d2-ca82ff2bb55e.png)
![minecraft_ windows 10 edition beta 06 01 2016 22_02_00](https://cloud.githubusercontent.com/assets/8733998/12155314/b19b5430-b4c4-11e5-9931-dc3899d090ab.png)
![minecraft_ windows 10 edition beta 06 01 2016 22_02_47](https://cloud.githubusercontent.com/assets/8733998/12155317/b1ad6742-b4c4-11e5-8c7c-c4ade781587d.png)
![minecraft_ windows 10 edition beta 06 01 2016 22_03_17](https://cloud.githubusercontent.com/assets/8733998/12155316/b1ab1d2a-b4c4-11e5-99d5-4c918bf2a28e.png)
![minecraft_ windows 10 edition beta 06 01 2016 22_04_29](https://cloud.githubusercontent.com/assets/8733998/12155315/b1a61ba4-b4c4-11e5-9198-bda3ef431a6e.png)
![minecraft_ windows 10 edition beta 06 01 2016 22_05_29](https://cloud.githubusercontent.com/assets/8733998/12155319/b1bc96cc-b4c4-11e5-8b30-889885d43796.png)
![minecraft_ windows 10 edition beta 06 01 2016 22_05_38](https://cloud.githubusercontent.com/assets/8733998/12155318/b1bc79d0-b4c4-11e5-943c-0339dc36358d.png)


Can you move that to a different file please ..


`change{Vote.count}.from(0).to(1)` is more robust if it can be used.


Put the pbx service states in the service documentation in configuration generated by the genDevConfig plugin OXE plugin. Put it in the Notes field (by memory), the one that was added to the Multisite UI.


Can you add an IDE test to make sure these are printed correctly?


How do we know this is the correct Vote?


We should test the content of this message.


Je ferais:
`
puts "Total Purchases: #{total_purchases}"
`
C'est plus idomatic


no to teraz jak ktoś pobierze i zacznie robić zmiany w pliku to trzeba się zgrywać żeby kolizji nie tworzyć bo nie wyobrażam sobie rozwiązywania ich w pliku z paradigma...


@palfrey For the `elixir-dev` image I added all dependencies necessary to build a simple phoenix application, including some extra erlang libs and rebar. How did you define the minimal dependencies for this image and which criteria do you think is reasonable? e.g. all dependencies to build a minimal cowboy app?

Thanks for contributing!

P.S. the `msaraiva/erlang:17.5` has been removed as described [here](https://github.com/msaraiva/alpine-erlang/issues/2). That means you don't need to add a Dockerfile for it anymore ;)


Will take care of that.


:tada: