fix(concatStatic): missing export creating breaking change

[ghetolay]

Description:

Export of concatStatic from operator/concat was removed on 5.5.
While I could easily replace :

import { concatStatic } from 'rxjs/operators/concat';

with

import { concat } from 'rxjs/observable/concat';

this is still a breaking change that could affect other projects.

[kwonoj]

This seems makes sense and we defer this change for v6 as breaking?

[rxjs-bot]

	Messages
📖

CJS: 2281.1KB, global: 748.1KB (gzipped: 120.4KB), min: 145.7KB (gzipped: 31.2KB)

Generated by 🚫 dangerJS

[coveralls]

Coverage increased (+0.0004%) to 97.399% when pulling 958ed80 on ghetolay:concatstatic_breakingchange into 24854cf on ReactiveX:master.

[ghetolay]

I looked a bit more on the code and this add inconsistency between how mergeStatic and concatStatic are handled.
mergeStatic is defined on operators/merge and reexported on both operator/merge and observable/merge while here concatStatic is defined on observable/concat, reexported on operator/concat and nothing inside operators/concat.

I was about to defined both inside observable/* and only reexport on operator/* for breaking change. But now that merge was also exported on operators/* in 5.5, removing it would create a new breaking change :)
So I'll just define them both inside observable/* and reexport on both operator/* and operators/*.
Will do that tomorrow cause I'm in need of some sleep right now 😴 .

[coveralls]

Coverage increased (+0.0007%) to 97.399% when pulling a06422e on ghetolay:concatstatic_breakingchange into 24854cf on ReactiveX:master.

[ghetolay]

I think this is good to go.
We'll just have to remove the exports on operator/* and operators/* on v6.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.