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

fix: start hovering default values of generic constants #15463

Conversation

ponyii
Copy link
Contributor

@ponyii ponyii commented Aug 15, 2023

It's just a kind of a postscriptum for my last PR adding default values of const generics to hir::ConstParamData. Here I patch other pieces of code which used to ignore const default values and which I managed to find (you're welcome to show me more)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 15, 2023
@ponyii ponyii changed the title start hovering default values of generic constants fix: start hovering default values of generic constants Aug 15, 2023
@HKalbasi
Copy link
Member

We should try to render consts using ast or in evaluated form instead of showing {const}, but since it is not from this PR let's merge this.
@bors r+

@bors
Copy link
Collaborator

bors commented Aug 15, 2023

📌 Commit fec5ff9 has been approved by HKalbasi

It is now in the queue for this repository.

@HKalbasi
Copy link
Member

@bors retry

@bors
Copy link
Collaborator

bors commented Aug 15, 2023

⌛ Testing commit fec5ff9 with merge 3461bfe...

bors added a commit that referenced this pull request Aug 15, 2023
…neric-const, r=HKalbasi

fix: start hovering default values of generic constants

It's just a kind of a postscriptum for [my last PR](#15179) adding default values of const generics to `hir::ConstParamData`. Here I patch other pieces of code which used to ignore const default values and which I managed to find (you're welcome to show me more)
@bors
Copy link
Collaborator

bors commented Aug 15, 2023

⌛ Testing commit fec5ff9 with merge b147709...

@bors
Copy link
Collaborator

bors commented Aug 15, 2023

☀️ Test successful - checks-actions
Approved by: HKalbasi
Pushing b147709 to master...

@bors bors merged commit b147709 into rust-lang:master Aug 15, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants