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

Use template-parameter and template parameter more consistently #7460

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cor3ntin
Copy link
Contributor

Try to use template-parameter only when we refer to a grammar construct, and to 'template parameter' everywhere else.

Adopt the same logic to template-argument/template argument.

This change might not be exhaustive.

The aim is to editorially adopt some of the wording changes made in P2841R5 to ease its review in core.

@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Dec 19, 2024
source/templates.tex Outdated Show resolved Hide resolved
source/templates.tex Outdated Show resolved Hide resolved
source/templates.tex Outdated Show resolved Hide resolved
source/templates.tex Outdated Show resolved Hide resolved
source/templates.tex Outdated Show resolved Hide resolved
Try to use template-parameter only when we refer to a
grammar construct, and to 'template parameter' everywhere else.

Adopt the same logic to template-argument/template argument.

This change might not be  exhaustive.

The aim is to editorially adopt some of the wording changes
made in P2841R5 to ease its review in core.
@cor3ntin cor3ntin force-pushed the corentin/template_argument_refactor branch from 2a492dd to 9fc22b5 Compare December 28, 2024 09:33
@cor3ntin cor3ntin requested a review from jensmaurer January 2, 2025 15:31
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Jan 2, 2025
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.

3 participants