Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Fancify readme #399

Merged
merged 3 commits into from
Jan 7, 2017
Merged

Fancify readme #399

merged 3 commits into from
Jan 7, 2017

Conversation

BinaryMuse
Copy link
Contributor

πŸ“ ✨

/cc @kuychaco @smashwilson for πŸ’­

Rendered

Michelle Tilley and others added 2 commits January 4, 2017 14:55
@kuychaco
Copy link
Contributor

kuychaco commented Jan 4, 2017

πŸ‘ looks great! just need to be sure we implement #400 before staff shipping

Copy link
Contributor

@smashwilson smashwilson left a comment

Choose a reason for hiding this comment

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

Looking great! Two minor comments from me πŸ™‡


**Settings View**

In settings view, click "Install," enter the URL to the repository in the search field, and press enter. In the installation card that appears, click "Install."
Copy link
Contributor

Choose a reason for hiding this comment

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

The text says "URL," but the screenshot uses a shortened org/repository form. Is there a consistent name for this?

Also: what's the minimum Atom/apm version necessary to install packages like this? Might be worth mentioning in case someone has fallen behind.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good question; Kat and I tackled that several months ago so it should be fine by now.

- [x] Create new branches
- [x] Push/pull/fetch
- [x] Abort merges
- [ ] Resolve merge conflicts via special UI
Copy link
Contributor

Choose a reason for hiding this comment

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

You could link out to #385 here too πŸ˜„

@BinaryMuse BinaryMuse merged commit cf08025 into master Jan 7, 2017
@BinaryMuse BinaryMuse deleted the mkt-fancify-readme branch January 7, 2017 00:00
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