Skip to content
This repository was archived by the owner on Apr 6, 2023. It is now read-only.

refactor: fix typo and code improvements#5344

Merged
pi0 merged 1 commit intomainfrom
chore/typo
Jun 8, 2022
Merged

refactor: fix typo and code improvements#5344
pi0 merged 1 commit intomainfrom
chore/typo

Conversation

@clarkdo
Copy link
Copy Markdown
Member

@clarkdo clarkdo commented Jun 8, 2022

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)

📚 Description

It's been a while since I changed framework code, so I'm taking some time to carefully understand the latest changes, and while reading, I found some typos and very minor changes, please feel free to close this pr if it's not relevant.

@clarkdo clarkdo requested a review from pi0 June 8, 2022 09:52
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 8, 2022

Deploy Preview for nuxt3-docs ready!

Name Link
🔨 Latest commit 06d7e6b
🔍 Latest deploy log https://app.netlify.com/sites/nuxt3-docs/deploys/62a07166ffc4e30009740af4
😎 Deploy Preview https://deploy-preview-5344--nuxt3-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

definition.meta.name = definition.meta.name || definition.name
// @ts-ignore
definition.meta.configKey = definition.meta.configKey || definition.configKey || definition.meta.name
definition.meta.configKey = definition.configKey || definition.meta.name
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Note: This is not a typo but to keep backward compatibility with the previous usage of the kit some modules were early adopted. But I'm okay with removing it now before 3.0.0

@pi0 pi0 changed the title chore: fix insufficient null check and typo refactor: fix typo and code improvements Jun 8, 2022
@pi0 pi0 merged commit 139fc11 into main Jun 8, 2022
@pi0 pi0 deleted the chore/typo branch June 8, 2022 19:49
@pi0 pi0 mentioned this pull request Jun 10, 2022
@danielroe danielroe added the 3.x label Jan 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants