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

Update README.md #255

Open
wants to merge 1 commit into
base: pit
Choose a base branch
from
Open

Update README.md #255

wants to merge 1 commit into from

Conversation

ashvin2005
Copy link

I have made a refined and visually engaging version of your README for Cherry On Tech’s website, with enhanced structure, clarity, and more emojis

This PR fixes...

Remove the example below. Then, add the related issue after the hash. Lastly, tell us what problems this PR is meant to solve.
Ex: This PR fixes #999. We didn't have a unicorn, so we went to the unicorn store and got one.

What I did...

Remove the example below. Then, list out the work you have done, in bullet form.
Ex:

  • one
  • two

How to test...

In a list format, tell us how to test this PR.
Ex:

  1. Navigate to https://example.com/cats
  2. Click on magic button
  3. See coolest cat

I learned...

Did you run into any interesting issues while making this PR? If you learned anything you would like to share, we'd love to know! Even if it is a fun fact or GIF!

I have made a refined and visually engaging version of your README for Cherry On Tech’s website, with enhanced structure, clarity, and more emojis
Copy link

netlify bot commented Oct 27, 2024

Deploy Preview for cherryontech ready!

Name Link
🔨 Latest commit 41f1840
🔍 Latest deploy log https://app.netlify.com/sites/cherryontech/deploys/671e554f4b8ac10008052876
😎 Deploy Preview https://deploy-preview-255--cherryontech.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 configuration.

Copy link
Contributor

@novellac novellac left a comment

Choose a reason for hiding this comment

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

This PR doesn't fix any issues, and it introduces a couple of new ones. If you'd like to contribute, we'd prefer you do things like fix, update, or create dictionary entries, as outlined in CONTRIBUTING.md.
However, I understand it's the end of Hacktoberfest, so I've left the PR open. You're welcome to make the requested changes and request another review. Best of luck with the rest of Hacktoberfest!

Comment on lines -20 to -22
## Contributing

We welcome you to contribute to our "Tech Dictionary," where we share tech terms from different _perspectives_. To get involved, read [CONTRIBUTING.md](CONTRIBUTING.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

Issue: This is a very important section and should not be removed.

Comment on lines -13 to -18
1. Make sure you have [node](https://nodejs.org/) installed. You can check by running `node --version`
1. Install Gridsome CLI tool if you don't have it: `npm install --global @gridsome/cli` or `yarn global add @gridsome/cli`
1. Clone the repo: `git clone https://github.com/cherryontech/website.git`
1. `cd website`
1. `npm install` or `yarn install`
1. Run `gridsome develop` to start a local dev server. In the terminal, you will see an address such as `http://localhost:8080/`. Navigate to that address in a browser to see the project in action.
Copy link
Contributor

Choose a reason for hiding this comment

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

Issue: The installation steps are important to remain here. If you'd like to enhance this section by moving the node check instructions (as you added on ln15), feel free to do that.

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