Skip to content
This repository has been archived by the owner on Aug 14, 2024. It is now read-only.

fix typo, carney has 1.4Pb not Tb #405

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

singhsaluja
Copy link
Contributor

Pull Request


PR Summary

Briefly describe the changes.

Check the types of changes present in this PR:

  • 🐛 Bug
  • 🐉 Feature
  • 🐸 Data (data folder - people/opportunities)
  • 🐶 Content (content folder)
  • 🐡 Other. Specify:

Checklist:

  • Commitizen was used for all commits.
  • Local site looks as expected after changes.
  • Contributing guidelines were followed.
  • If changes affect development, was the README updated?
If PR to production

@mcmcgrath13 mcmcgrath13 temporarily deployed to ccv-website-pr-405 September 30, 2021 15:41 Inactive
@github-actions
Copy link

Technical Reviewer Checklist

Make sure the updates follow the contribution guidelines and the code is clear. Some of the things you should check:

  • All commits use conventional style
  • Website builds
  • Code is readable and easily understandable, with comments, but not commented out code
  • Spacing and indentation is constistent
  • README was updated (if needed) and update is comprehensive and useful
  • No extraneous files added, especially a .env or similar

@github-actions
Copy link

Content Reviewer Checklist

The content reviewer should have the experience/background to vet the content changes to this Pull Request.

Make sure the updates are consistent with the CCV website as a whole, the content is in the right place, and is easily understandable/accessible. Some of the things you should check:

  • Check spelling
  • Will this make sense to an outsider?
  • Is this information redundant with other places in the website?
  • Do links work and go to the expected place?
  • Do the changes look how you would expect?

@paulstey
Copy link
Member

Hi @singhsaluja!! Thanks for fixing this! Would you mind pushing one more time, though? There is a typo in the word Petabyte at the moment.

@mcmcgrath13 mcmcgrath13 temporarily deployed to ccv-website-pr-405 September 30, 2021 16:10 Inactive
Copy link
Member

@paulstey paulstey left a comment

Choose a reason for hiding this comment

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

Thank you, @singhsaluja!!!

@mcmcgrath13 mcmcgrath13 temporarily deployed to ccv-website-pr-405 October 19, 2021 19:47 Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants