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

docs: updates to make styling consistent #36

Merged
merged 26 commits into from
Nov 28, 2022

Conversation

naman-tiwari
Copy link
Contributor

  • Enhanced the readme file by adding emojis.

@netlify
Copy link

netlify bot commented May 25, 2022

Deploy Preview for enchanting-cascaron-aa83e8 failed.

Name Link
🔨 Latest commit 50db191
🔍 Latest deploy log https://app.netlify.com/sites/enchanting-cascaron-aa83e8/deploys/63844952f2c6e8000902df68

Copy link
Collaborator

@magicmatatjahu magicmatatjahu left a comment

Choose a reason for hiding this comment

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

Please try to make good indents for code:

  ```sh
  npm install

README.md Outdated

```cmd
npm install --save-dev
npm install --save-dev
Copy link
Collaborator

Choose a reason for hiding this comment

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

Try to make good indents by adding two spaces before "`" signs and also change

npm install --save-dev

to the

npm install

added indents before npm commands and changed ```npm install --save-dev``` to ```npm install```
@naman-tiwari
Copy link
Contributor Author

I have added indents before npm commands and changed npm install --save-dev to npm install

@mcturco
Copy link
Collaborator

mcturco commented May 26, 2022

I think my suggestions that I wrote in the other PR got lost. I think it was just suggesting to re-write some of the wording so it reads better. Are you able to see my review in the other PR and apply the changes here, @naman-tiwari?

@naman-tiwari
Copy link
Contributor Author

I think my suggestions that I wrote in the other PR got lost. I think it was just suggesting to re-write some of the wording so it reads better. Are you able to see my review in the other PR and apply the changes here, @naman-tiwari?

I will resolve the requested changes by today @mcturco .

Copy link
Contributor Author

@naman-tiwari naman-tiwari left a comment

Choose a reason for hiding this comment

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

The requested changes by @mcturco have been addressed.

README.md Outdated

```cmd
npm run storybook
```

- This will start your local Storybook environment on port `6006` by default.
- By default, this will launch your local Storybook environment on port '6006'.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- By default, this will launch your local Storybook environment on port '6006'.
- By default, this will launch your local Storybook environment on port `6006`.

README.md Outdated

```cmd
npm install
```

- Now, as you have all the `node_modules` installed in your project, you can start the Storybook environment. Run the command below to start it:
- Now that you've installed all of the 'node modules' in your project, you can launch the Storybook environment. To get started, type the command below:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Now that you've installed all of the 'node modules' in your project, you can launch the Storybook environment. To get started, type the command below:
- Now that you've installed all of the `node modules` in your project, you can launch the Storybook environment. To get started, type the command below:

reversed quotation marked on lines 13 and 19
Copy link
Collaborator

@magicmatatjahu magicmatatjahu left a comment

Choose a reason for hiding this comment

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

LGTM! @mcturco Could you check that PR? :)

Copy link
Collaborator

@mcturco mcturco left a comment

Choose a reason for hiding this comment

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

LGTM! 🚢

@magicmatatjahu
Copy link
Collaborator

@naman-tiwari Sorry for such a delay. As I see in the netlify logs you should remove export script from build script and we can merge it :)

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Nov 23, 2022
@fmvilas
Copy link
Collaborator

fmvilas commented Nov 28, 2022

/rtm

@asyncapi-bot asyncapi-bot merged commit ade8219 into asyncapi-archived-repos:master Nov 28, 2022
@fmvilas
Copy link
Collaborator

fmvilas commented Nov 28, 2022

I'm merging as the problem is not related to this PR anyway.

@fmvilas
Copy link
Collaborator

fmvilas commented Nov 28, 2022

Sorry it took that long @naman-tiwari. I just noticed it because the bot added the stale label.

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.

None yet

5 participants