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

Add code excerpts to new type promotion failures #5359

Merged
merged 1 commit into from
Jun 21, 2024
Merged

Conversation

MaryaBelanger
Copy link
Contributor

@MaryaBelanger MaryaBelanger commented Nov 17, 2023

Excerpts added in #5246

Fixes #5350

@MaryaBelanger
Copy link
Contributor Author

I'm still not sure how to implement the excerpts for the {:.bad} examples, tbd. Maybe the file just has a bunch of ignore_for_file conditions?

I know there's a way to print the actual error from the excerpts too (I think the Fixing common type problems page does it), so tbd on that too.

@parlough parlough self-requested a review November 18, 2023 02:19
@MaryaBelanger MaryaBelanger self-assigned this Dec 1, 2023
@atsansone atsansone requested a review from craiglabenz January 17, 2024 22:36
@parlough parlough removed the request for review from craiglabenz January 23, 2024 20:36
@parlough parlough assigned parlough and unassigned MaryaBelanger Jan 23, 2024
@dart-github-bot
Copy link
Collaborator

dart-github-bot commented Feb 4, 2024

Visit the preview URL for this PR (updated for commit f722e2c):

https://dart-dev--pr5359-promo-code-exc-lmxwqb8m.web.app

@dart-lang dart-lang deleted a comment from github-actions bot Feb 11, 2024
@atsansone atsansone assigned MaryaBelanger and unassigned parlough Mar 6, 2024
@parlough parlough assigned parlough and unassigned MaryaBelanger May 22, 2024
parlough added a commit that referenced this pull request May 31, 2024
- Minor copy fixes
- Limit textual lines to 80 characters
- Add some missing return types to code snippets
- Place link definitions near the usage of them
- Use code font in a few more places where appropriate
- Replace curly quotation marks with normal ones

Landing this first to make finalizing
#5359 easier.
@atsansone atsansone assigned MaryaBelanger and unassigned parlough Jun 3, 2024
@atsansone
Copy link
Contributor

@MaryaBelanger : Once you get this PR updated, I'll take another look.

@atsansone
Copy link
Contributor

@MaryaBelanger : This was last touched by you in 2023. Do you think you can come back to this or should someone else pick it up?

@parlough
Copy link
Member

I took this one over and will try to get back to it this week.

Co-authored-by: Marya Belanger <mbelanger@google.com>
Copy link
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

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

Thanks for doing this @MaryaBelanger, looks great to me!

I pulled out some of the copy fixes and clean up out into #5859, then rebased on top of that cleanup, and finally fixed the few remaining errors.

@parlough parlough merged commit 8288d78 into main Jun 21, 2024
10 checks passed
@parlough parlough deleted the promo-code-exc branch June 21, 2024 05:57
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.2] Add code excerpts to non-promotion-reasons
4 participants