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

Replace the glossary entry for override inference #2654

Open
bwilkerson opened this issue Sep 25, 2020 · 5 comments
Open

Replace the glossary entry for override inference #2654

bwilkerson opened this issue Sep 25, 2020 · 5 comments
Labels
a.glossary Relates to unified list of terms and definitions dev.null-safety Relates to transforming or migrating Dart code to sound null safety e1-hours Can complete in < 8 hours of normal, not dedicated, work p2-medium Necessary but not urgent concern. Resolve when possible. st.triage.ltw Indicates Lead Tech Writer has triaged t.diagnostics Relates to diagnostics, analysis, or linting of code

Comments

@bwilkerson
Copy link
Member

The diagnostic messages documentation (https://dart.dev/tools/diagnostic-messages) contains a glossary entry for override inference. The text reads more like expository text than a glossary entry. I think we'll need to have documentation for Null Safety that explains how type inference in general, and override inference in particular, works, so it might be good to replace the glossary entry with a pointer to the inference documentation (after it exists).

@bwilkerson
Copy link
Member Author

@domesticmouse

@kwalrath kwalrath added docs e1-hours Can complete in < 8 hours of normal, not dedicated, work dev.null-safety Relates to transforming or migrating Dart code to sound null safety p1-high Major but not urgent concern: Resolve in months. Update each month. labels Sep 25, 2020
@kwalrath kwalrath added this to the Null safety: beta release milestone Sep 25, 2020
@domesticmouse
Copy link
Member

I'm happy to help refine, but I lack the knowledge to write the original text. Help?

@bwilkerson
Copy link
Member Author

(I probably should have said, but the referenced page hasn't actually been updated yet. The glossary entry is visible in this PR: #2655.)

I'm willing to expand on what I wrote for the glossary entry, but @munificent could probably do a better job, both with override inference and with other forms of inference.

@kwalrath kwalrath added p2-medium Necessary but not urgent concern. Resolve when possible. and removed p1-high Major but not urgent concern: Resolve in months. Update each month. labels Oct 29, 2020
@kwalrath kwalrath removed this from the Null safety: beta release milestone Nov 19, 2020
@atsansone atsansone added a.glossary Relates to unified list of terms and definitions st.triage.ltw Indicates Lead Tech Writer has triaged t.diagnostics Relates to diagnostics, analysis, or linting of code and removed docs labels Apr 5, 2023
@atsansone
Copy link
Contributor

Updated in #2662. Closing.

@parlough
Copy link
Member

parlough commented Apr 6, 2023

I'm going to reopen this one. That just introduced the glossary entry. The idea here is to instead link to proper documentation on override inference (and type inference in general) when we finally get to expanding on that.

Related issues: #4745

@parlough parlough reopened this Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a.glossary Relates to unified list of terms and definitions dev.null-safety Relates to transforming or migrating Dart code to sound null safety e1-hours Can complete in < 8 hours of normal, not dedicated, work p2-medium Necessary but not urgent concern. Resolve when possible. st.triage.ltw Indicates Lead Tech Writer has triaged t.diagnostics Relates to diagnostics, analysis, or linting of code
Projects
None yet
Development

No branches or pull requests

5 participants