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

Rename source.map => transformValuesWith and add warning #4

Closed
wants to merge 39 commits into from
Closed

Rename source.map => transformValuesWith and add warning #4

wants to merge 39 commits into from

Conversation

natsukagami
Copy link

See warning

natsukagami and others added 30 commits December 8, 2023 19:39
@natsukagami
Set up formatting configuration
11409b8
@natsukagami
Do the actual formatting
5b07b3d
@natsukagami
Enable CI formatting check
e7833da
@natsukagami
Merge pull request lampepfl#22 from natsukagami/formatting
1137ee0 
Enable formatting with `scalafmt`
@natsukagami
Fancy impl of channels based on lockBoth
d33dfc7
@natsukagami
Add shorthand await syntax to sources
107a8cd
@natsukagami
Channels returning inner classes to make fancy select syntax
7e22f5c
@natsukagami
Remove suspendSupport requirement from channel
3b72849
@natsukagami
Implement unbounded channels
85e4ec2
@natsukagami
Implement collectors
afb7b89
@natsukagami
Reorder things a bit
f350f42
@natsukagami
Implement collector-based future collection extensions
af52574
@natsukagami
Some renaming and documentations
e7eec22
@natsukagami
Add Source.values
40c6490
@natsukagami
Make awaitAll throw as soon as possible
d9c5d48
@natsukagami
Remove inner classes and rewrite ChannelMultiplexer
8f41bc3 
... so that it no longer creates "spooky actions at a distance"
@m8nmueller
minor remarks
25a1300
@natsukagami
Add comments in channel impl
8f3500c
@natsukagami
Minor fixed in Source.values
7c26d1e
@m8nmueller
two more eyes on channels: fixes
b7affc8
@natsukagami
Address review comments
ff88958
@m8nmueller
minor channel cleanup/renaming
3fffabc
@natsukagami
Add CountDownLatch to ChannelMultiplexer test
09bc8c1 
ChannelMultiplexer is very dependent on subscribing order, and should be
either set up correctly or used by subscribers that don't really care
about past events.
@natsukagami
Rewrite channel multiplexer test to follow create/wire/run stages
be5a125
@natsukagami
Rename Timer.start to run
b96497d 
... to make it consistent with ChannelMultiplexer.
Overall `start` sounds more like a background running thing w.r.t
threads.
@natsukagami
Merge pull request lampepfl#23 from natsukagami/explicit-channels-prelim
86159db 
Channels implementation with new listeners
@natsukagami
Rename result => awaitResult, value => await, make them Source methods
17f5bc8
@natsukagami
use more efficient alt/altCancel
7e3b0f7
@natsukagami
Slight improvement of the tests
81b67f7
@natsukagami
Guarantee that future cancellation is done once
05f8e8b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@natsukagami @m8nmueller