Skip to content

implemented range adaptors#89

Merged
liss-h merged 7 commits into
developfrom
feature/more-range-addaptors
Jun 11, 2025
Merged

implemented range adaptors#89
liss-h merged 7 commits into
developfrom
feature/more-range-addaptors

Conversation

@bigerl

@bigerl bigerl commented Jun 6, 2025

Copy link
Copy Markdown
Member

Implements the remaining range adaptors from #86 except for join.

I would like to get some feedback on the code before continuing with the examples and readme.

@bigerl bigerl linked an issue Jun 6, 2025 that may be closed by this pull request
6 tasks
@bigerl bigerl removed a link to an issue Jun 6, 2025
6 tasks

@liss-h liss-h left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, I found a lot of code convention issues

Comment thread include/dice/template-library/ranges.hpp
Comment thread include/dice/template-library/ranges.hpp Outdated
Comment thread include/dice/template-library/ranges.hpp Outdated
Comment thread include/dice/template-library/ranges.hpp Outdated
Comment thread include/dice/template-library/ranges.hpp Outdated
Comment thread include/dice/template-library/ranges.hpp Outdated
Comment thread include/dice/template-library/ranges.hpp
Comment thread include/dice/template-library/ranges.hpp Outdated
Comment thread include/dice/template-library/ranges.hpp Outdated
Comment thread tests/tests_ranges.cpp
@bigerl bigerl changed the base branch from feature/squashing-range-addaptors to develop June 10, 2025 09:16
@bigerl bigerl changed the title implemented missing ranges (except for join) implemented range adaptors Jun 10, 2025
@bigerl bigerl requested a review from liss-h June 10, 2025 13:24

@liss-h liss-h left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I pushed a commit to clean up some things, please review the changes. Additionally, I have some more comments

Comment thread README.md Outdated
Comment thread examples/example_ranges.cpp
Comment thread include/dice/template-library/ranges.hpp
Comment thread include/dice/template-library/ranges.hpp
@liss-h liss-h merged commit a550dd4 into develop Jun 11, 2025
7 checks passed
@liss-h liss-h mentioned this pull request Jun 11, 2025
6 tasks
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.

2 participants