Skip to content

docs(skills): fix small typos in skill references#412

Closed
AllDmeat wants to merge 1 commit into
microsoft:mainfrom
AllDmeat:fix/typos-in-skill-references
Closed

docs(skills): fix small typos in skill references#412
AllDmeat wants to merge 1 commit into
microsoft:mainfrom
AllDmeat:fix/typos-in-skill-references

Conversation

@AllDmeat
Copy link
Copy Markdown

Found a few small issues in skills/playwright-cli/references/* while vendoring the skills into another project and getting them reviewed:

  • spec-driven-testing.md — example test group name was 'Singing in and out', changed to 'Signing in and out'.
  • storage-state.md — both the expires example in the storage-state JSON (line 41) and the --expires example for cookie-set (line 95) used Unix timestamp 1735689600, which is Jan 1 2025 — already in the past, making the cookie examples read as expired. Bumped both to 1893456000 (Jan 1 2030).
  • video-recording.md — "pulling appropriate pauses between the actions" → "inserting appropriate pauses".

No behavior changes — docs only.

@AllDmeat
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree company="Dodo Engineering"

- spec-driven-testing.md: `Singing in and out` → `Signing in and out`
- storage-state.md: bump example cookie `expires` from 1735689600
  (Jan 1, 2025) to 1893456000 (Jan 1, 2030) so the example doesn't
  read as already-expired
- video-recording.md: `pulling appropriate pauses` → `inserting
  appropriate pauses`
@AllDmeat AllDmeat force-pushed the fix/typos-in-skill-references branch from b1e7f02 to f626401 Compare May 21, 2026 07:28
@dgozman
Copy link
Copy Markdown
Contributor

dgozman commented May 21, 2026

@AllDmeat Thank you for the PR! The source of truth for this skill lives in playwright repository, and is periodically rolled into here. Could you please open a PR against playwright instead? Thank you!

@AllDmeat
Copy link
Copy Markdown
Author

Thanks for the pointer @dgozman! Reopened against playwright: microsoft/playwright#40942 — closing this one.

@AllDmeat AllDmeat closed this May 21, 2026
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.

2 participants