Skip to content

Fix minor typo in live directive paragraph#769

Merged
arthurevans merged 1 commit intolit:mainfrom
talhaguy:live-directive-typo
May 9, 2022
Merged

Fix minor typo in live directive paragraph#769
arthurevans merged 1 commit intolit:mainfrom
talhaguy:live-directive-typo

Conversation

@talhaguy
Copy link
Copy Markdown
Contributor

Fixing a minor typo in the live directive description. Changed agains to against.

PS - When doing a search across files, I saw that there are two more instances of the typo (below). However, I wasn't sure how to rebuild these. I tried running npm run build from /packages/lit-dev-api/package.json but it didn't seem to work. I can gladly update this PR if I can get some guidance on how to do so!

  • /packages/lit-dev-api/api-data/lit-2/pages.json
  • /packages/lit-dev-api/api-data/lit-html-1/pages.json

@arthurevans
Copy link
Copy Markdown
Contributor

Thanks for the catch! These pages are actually generated from the source code (current docs from the main branch, 1.x docs from the lit-html-1.x branch):

https://github.com/lit/lit/blob/main/packages/lit-html/src/directives/live.ts#L86
https://github.com/lit/lit/blob/lit-html-1.x/src/directives/live.ts#L33

Do you want to make PRs against the lit repo? If that's too much work, let me know.

@arthurevans arthurevans merged commit bc1141e into lit:main May 9, 2022
@arthurevans
Copy link
Copy Markdown
Contributor

Thanks for the fix!

@arthurevans arthurevans mentioned this pull request May 9, 2022
arthurevans pushed a commit to lit/lit that referenced this pull request May 9, 2022
Fix is API doc only, error originally reported in lit/lit.dev#769.
arthurevans pushed a commit to lit/lit that referenced this pull request May 10, 2022
Fix is API doc only, error originally reported in lit/lit.dev#769.
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