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.

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

[ranges.syn] Add \ref for ref_view #5652

Merged
merged 3 commits into from Jul 26, 2022
Merged

[ranges.syn] Add \ref for ref_view #5652

merged 3 commits into from Jul 26, 2022

Conversation

hewillk
Copy link
Contributor

@hewillk hewillk commented Jul 26, 2022

No description provided.

Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

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

Looks mostly good.

Please change the commit description

[ranges.syn] Add template name for take_view

to say "... template parameter name..."

(Having two pull requests for these two unrelated changes would have allowed me to fix the commit description in the github GUI, easily).

@hewillk
Copy link
Contributor Author

hewillk commented Jul 26, 2022

Reverted the take_view template parameter name change. I'd like to save this part for the next pull request.

@jensmaurer
Copy link
Member

(oh, and force-pushing is fine for pull requests)

@hewillk hewillk changed the title [ranges.syn] Add \ref for ref_view and template name for take_view [ranges.syn] Add \ref for ref_view Jul 26, 2022
@jensmaurer jensmaurer merged commit f6cb844 into cplusplus:main Jul 26, 2022
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.

None yet

2 participants