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

extend chalk TypeName with builtin types #368

Closed
18 tasks done
nikomatsakis opened this issue Mar 31, 2020 · 15 comments
Closed
18 tasks done

extend chalk TypeName with builtin types #368

nikomatsakis opened this issue Mar 31, 2020 · 15 comments
Labels
current-sprint Being worked on in the current sprint

Comments

@nikomatsakis
Copy link
Contributor

nikomatsakis commented Mar 31, 2020

The TypeName enum needs to be extended to cover the "built-in" types from Rust. The following variants would eventually want to be added:

(Need to update this list, not yet complete, see this section of the chalk book for a more complete list as well.

@MarkMcCaskey
Copy link
Contributor

I'm interested in taking this on! I'll follow up with a PR

@MarkMcCaskey
Copy link
Contributor

@rustbot claim

@basil-cow
Copy link
Contributor

@MarkMcCaskey what's the status on this?

@MarkMcCaskey
Copy link
Contributor

Sorry about the radio silence -- I've been busier than I anticipated recently and haven't had the spare focus to work on this. I haven't worked on closures or functions other than what was in the original PR yet. I can dedicate a bit of time to it this weekend and more time to it next week. If this is blocking other issues and/or if someone wants to implement it, I have no problem with someone else taking it on but I'd be happy to work on it otherwise!

nikomatsakis added a commit to nikomatsakis/wg-traits that referenced this issue May 4, 2020
nikomatsakis added a commit to nikomatsakis/wg-traits that referenced this issue May 5, 2020
nikomatsakis added a commit to rust-lang/types-team that referenced this issue May 5, 2020
@nikomatsakis
Copy link
Contributor Author

@rustbot release-assignment

I think @MarkMcCaskey is busy, but also I'm going to break this into distinct issues.

@nikomatsakis
Copy link
Contributor Author

OK, I broke out a number of issues thus far that people can pick up.

@nikomatsakis
Copy link
Contributor Author

@rustbot release-assignment

This is a tracking issue, so it doesn't really need an assignee.

@vringar
Copy link

vringar commented Sep 3, 2020

Could this issue be closed and the remaining items moved into #16 directly so there's a more centralized overview of what still needs to be done for a shared type library?

@AzureMarker
Copy link
Member

@vringar Not sure if you linked to the right issue; that one is about checking kinds during lowering, from 2017.

@jackh726
Copy link
Member

jackh726 commented Sep 4, 2020

@vringar probably means rust-lang/types-team#16

I think keeping this open and linking to this from there is the best idea. Ultimately, yeah, there's not much left to here; but, I don't think it makes sense to put all the finer details there.

@vringar
Copy link

vringar commented Sep 4, 2020

Sorry for fucking up the linking.
@jackh726 is right is the assumption but also in the resolution.
Sorry for creating noise.

@jackh726
Copy link
Member

jackh726 commented Sep 4, 2020

@vringar No worries!

@Aaron1011
Copy link
Member

With PR #593, we now have Generator and GeneratorWitness. All of the types in this issue are now checked off - is there anything left to do here?

@jackh726
Copy link
Member

jackh726 commented Oct 3, 2020

@Aaron1011 nothing left to do here

Thanks everyone for all the work to get this done :)

@jackh726 jackh726 closed this as completed Oct 3, 2020
@LeSeulArtichaut
Copy link
Contributor

LeSeulArtichaut commented Oct 19, 2020

Should the work item associated to this issue in rust-lang/types-team#16 be checked?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
current-sprint Being worked on in the current sprint
Projects
None yet
Development

No branches or pull requests

8 participants