Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove public scope from Flux operator #1036

Closed
smaldini opened this issue Jan 23, 2018 · 0 comments
Closed

Remove public scope from Flux operator #1036

smaldini opened this issue Jan 23, 2018 · 0 comments
Labels
type/bug A general bug warn/api-change Breaking change with compilation errors
Milestone
3.1.4.RELEASE

Comments

@smaldini
Copy link
Contributor

The recent releases have inadvertently opened 3 operators to the public API. These API should not be used and will be package scoped.

  • FluxDelaySequence
  • FluxIndex
  • FluxIndexFuseable
@smaldini smaldini added type/bug A general bug warn/api-change Breaking change with compilation errors labels Jan 23, 2018
@smaldini smaldini added this to the 3.1.4.RELEASE milestone Jan 23, 2018
simonbasle added a commit that referenced this issue Jan 23, 2018
@simonbasle
fix #1036 Remove erroneous public scope from 3 internal operator classes
2c6d436 
FluxDelaySequence, FluxIndex and FluxIndexFuseable are internal
implementations and should have been package-private instead of public.
simonbasle added a commit that referenced this issue Jan 23, 2018
@simonbasle
fix #1036 Remove erroneous public scope from 3 internal operator classes
27699cc 
FluxDelaySequence, FluxIndex and FluxIndexFuseable are internal
implementations and should have been package-private instead of public.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/bug A general bug warn/api-change Breaking change with compilation errors
Projects
None yet
Milestone
3.1.4.RELEASE
Development

No branches or pull requests
1 participant
@smaldini