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

fix: multiple typos of different importance #2629

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

crStiv
Copy link

@crStiv crStiv commented Dec 17, 2024

Description

This pull request addresses multiple typos across several files to improve clarity, correctness, and consistency within the codebase and documentation. The changes include fixes in comments, error messages, and test descriptions.

Fixes:

  • deploy/k8s/README.md:
    • Fixed typo archievearchive.
  • packages/cli/CHANGELOG.md:
    • Fixed typo deployemntdeployment.
  • packages/cli/src/commands/deployment/index.ts:
    • Fixed typo an deploymenta deployment.
  • packages/cli/src/commands/migrate.test.ts:
    • Fixed typo IntergrationIntegration.
  • packages/cli/src/commands/publish.test.ts:
    • Fixed typo IntergrationIntegration.
    • Fixed typo manfiestmanifest.

Type of change

Please delete options that are not relevant:

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have tested locally
  • I have performed a self review of my changes
  • Updated any relevant documentation
  • Linked to any relevant issues
  • I have added tests relevant to my changes
  • Any dependent changes have been merged and published in downstream modules
  • My code is up to date with the base branch
  • I have updated relevant changelogs. We suggest using chan

Allow edits by maintainers
☑️

@stwiname
Copy link
Collaborator

Can you please update the changelogs in relevant packages.

Fixed typo archieve → archive
Fixed typo deployemnt → deployment
@crStiv
Copy link
Author

crStiv commented Dec 18, 2024

Can you please update the changelogs in relevant packages.

I'm not sure what did you mean, you want me to change comments for every commit included in this PR?

Fixed typo an deployment → a deployment
Fixed typo Intergration → Integration.
Fixed typo Intergration → Integration.
Fixed typo manfiest → manifest.
@stwiname
Copy link
Collaborator

Can you please update the changelogs in relevant packages.

I'm not sure what did you mean, you want me to change comments for every commit included in this PR?

You've ticked this box but not done so, if you want examples look at other PRs
[ ] I have updated relevant changelogs. We suggest using chan

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