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

Revamped the README #1915

Merged
merged 2 commits into from
Feb 3, 2017
Merged

Revamped the README #1915

merged 2 commits into from
Feb 3, 2017

Conversation

clarkbw
Copy link
Contributor

@clarkbw clarkbw commented Feb 3, 2017

  • Mostly lots of little fixes to the links
  • Updated the last sentence of the first second paragraph
  • Updated the debugger screenshot
  • Changed ‘Getting Setup’ to ‘Quick Setup’

* Mostly lots of little fixes to the links
* Updated the last sentence of the first second paragraph
* Updated the debugger screenshot
* Changed ‘Getting Setup’ to ‘Quick Setup’
README.md Outdated
yarn start
```

#### Next Steps

* Claim an [up for grabs][up-for-grabs] issues. If you get stuck, we'd be happy to [help].
* Claim an [up-for-grabs] issues. If you get stuck, we'd be happy to [help].
Copy link
Contributor

Choose a reason for hiding this comment

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

issues -> issue here?

<a href="https://devtools-html.slack.com/messages/general/">
<img src="https://devtools-html-slack.herokuapp.com/badge.svg">
</a>
[![slack-badge]][slack]
Copy link
Contributor

Choose a reason for hiding this comment

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

haha - not seeing the button. This is really cool.you're taking these links to the next level

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Perhaps too far? 😏

It does ensure we're using the right link everywhere, previously this pointed to https://devtools-html.slack.com/messages/general/ instead of the heroku page https://devtools-html-slack.herokuapp.com/ and I think that's the right way to point people.

Which also reminds me that we should fork the heroku app into devtools-html so we can customize it as needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But I'm not sure why the diff doesn't handle those well.

@jasonLaster jasonLaster merged commit 79fc861 into firefox-devtools:master Feb 3, 2017
@clarkbw clarkbw deleted the update-readme branch February 3, 2017 17:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants