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

Minor English-language improvements in installation.md #5805

Merged
merged 1 commit into from
Mar 5, 2023

Conversation

saifkhichi96
Copy link
Contributor

Motivation and context

There are some poorly written sentences in documentation pages. For example:

You can use a share storage for data uploading during you are creating a task. To do that you can mount it to CVAT docker container.

These make the documentation look a little informal, and since Installation Guide is probably the first page most new CVAT users visit, I updated some of the sentences on this page to have a better structure / sound more formal.

How has this been tested?

This change did not require any tests.

Checklist

  • I submit my changes into the develop branch
  • I have added a description of my changes into the CHANGELOG file
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • I have linked related issues (see GitHub docs)
  • I have increased versions of npm packages if it is necessary
    (cvat-canvas,
    cvat-core,
    cvat-data and
    cvat-ui)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

Copy link
Contributor

@nmanovic nmanovic left a comment

Choose a reason for hiding this comment

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

@saifkhichi96 , thanks for the contribution!

@nmanovic nmanovic merged commit 7ecda9f into cvat-ai:develop Mar 5, 2023
@saifkhichi96 saifkhichi96 deleted the patch-1 branch March 6, 2023 12:13
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