Skip to content

Update README.md#255

Closed
ashvin2005 wants to merge 1 commit intocherryontech:pitfrom
ashvin2005:patch-1
Closed

Update README.md#255
ashvin2005 wants to merge 1 commit intocherryontech:pitfrom
ashvin2005:patch-1

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
@netlify
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.

@alliequintano
Copy link
Collaborator

Closing because this PR is old.

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.

3 participants