Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Update readme #60

Closed
wants to merge 11 commits into from
Closed

Update readme #60

wants to merge 11 commits into from

Conversation

baw
Copy link
Contributor

@baw baw commented Jul 31, 2015

I have added some information about how to use the modules to the README and moved some of the module descriptions to there only files under the docs/ folder and added some examples to improve ease of use. I tried to also add some of the modules that I felt would be more commonly used as to not have the README get to long.

closes #50

Brian Weiser added 7 commits July 21, 2015 11:45
pulled out module details into separate files
* also began adding examples of module exports
updated usage information

### Feature Detection
This package is currently in Alpha with a initial stable release scheduled for later this year. You can install the Alpha
through npm:
Copy link
Member

Choose a reason for hiding this comment

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

The dojo2 package currently available through npm is not this package. Cloning the repo is currently the only way to go.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am pretty certain that at least that command does install a compiled version (although, outdated and probably never to be updated) since that alpha version does point to @bryanforbes core repo.

@jason0x43
Copy link
Member

Overall the organization is good, but it could be made a bit more consistent. The initial "Installation" and "Dojo Usage" sections should probably be combined, and the usage instructions could be made a bit clearer.

@baw
Copy link
Contributor Author

baw commented Aug 21, 2015

I've updated the branch based on feedback.

@kitsonk
Copy link
Member

kitsonk commented Nov 16, 2015

Merged via 9becfcf

@kitsonk kitsonk closed this Nov 16, 2015
@kfranqueiro
Copy link
Member

I'm not sure how heavily this was reviewed before it was merged, but at minimum I would at least suggest changing the headings in the various specific docs/*md files so that they only include the actual name of the API, with a sentence describing it underneath.

e.g., change:

Name - does a thing

to

Name

Does a thing.

(I haven't had time to look in depth as to whether there are any factual issues or omissions in the PR.)

@dylans dylans added this to the Milestone 2 milestone Jan 10, 2018
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.

README Update
5 participants