Skip to content

Problem: records by themselves are not very useful#2

Merged
yrashk merged 1 commit into
sit-fyi:masterfrom
yrashk:reducers
Jan 29, 2018
Merged

Problem: records by themselves are not very useful#2
yrashk merged 1 commit into
sit-fyi:masterfrom
yrashk:reducers

Conversation

@yrashk
Copy link
Copy Markdown
Member

@yrashk yrashk commented Jan 29, 2018

It's hard to figure out current issue's state by
just lookin at records

Solution: introduce a concept of reducers that help
building representation based on records.

It's hard to figure out current issue's state by
just lookin at records

Solution: introduce a concept of reducers that help
building representation based on records.
@yrashk yrashk merged commit 836dc4d into sit-fyi:master Jan 29, 2018
yrashk added a commit that referenced this pull request Feb 26, 2018
```
   --> sit-web/src/webapp.rs:202:32
       |
   202 |             let record = match sit_core::repository::IssueRecordIter::flatten(issue.record_iter().unwrap()).find(|r| r.encoded_hash() == record) {
       |                                                                       ^^^^^^^ multiple `flatten` found
       |
       = note: candidate #1 is defined in an impl of the trait `std::iter::Iterator` for the type `sit_core::repository::IssueRecordIter<'_>`
       = note: candidate #2 is defined in an impl of the trait `itertools::Itertools` for the type `_`
       = note: candidate #3 is defined in the trait `rayon::iter::ParallelIterator`
```

This is, perhaps, related to the addition of the [`iterator_flatten`
nightly feature](rust-lang/rust#48213).

Solution: use itertools' version explicitly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant