Skip to content

Conversation

@Jeevankumar-s
Copy link
Contributor

Checklist:

Closes #65331

This updates Step 10 of the Quincy’s Job Tips workshop to use the — HTML entity consistently across the description, seed content, and tests..

@Jeevankumar-s Jeevankumar-s requested review from a team as code owners January 26, 2026 12:45
@github-actions github-actions bot added the scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. label Jan 26, 2026
@majestic-owl448
Copy link
Contributor

tests are failing, double check which test is it and please adjust it

@Jeevankumar-s
Copy link
Contributor Author

Jeevankumar-s commented Jan 26, 2026

tests are failing, double check which test is it and please adjust it

@majestic-owl448 Thanks for the review! I rechecked the failing test and realized the issue is the reverse of what I thought.

The test uses innerText, which returns rendered text, not HTML. When — is rendered, it becomes the em dash character (—), so innerText will never return —.

The fix is to update the test assertion to check for instead of —. The seed content and description should still keep —.

I’ll update the PR with this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

em dash issue in workshop-quincys-job-tips

2 participants