Skip to content
This repository has been archived by the owner on Nov 17, 2022. It is now read-only.

Update README #198

Merged
merged 9 commits into from
Oct 24, 2017
Merged

Update README #198

merged 9 commits into from
Oct 24, 2017

Conversation

hayfield
Copy link
Contributor

Various small fixes and improvements to the README.

iati.core -> pyIATI
@hayfield hayfield added docs Changes to documentation. incomplete A PR that is in a state that is not ready for review. labels Oct 23, 2017
Having info about them prominently, then having them
look a bit dud isn't great. Adding a warning partially
helps with this problem.
There should only be single spaces between sentences.
Leaving this here adds additional maintenance burden, and
this is not undertaken.

The Issues are used for this purpose, and are able to adapt
over time, with discussions and additional information being
added.

The current situation of using Issues isn't ideal, though
it's not immediately obvious what a correct solution is.
Some information had become out-of-date. This brings it back in line
with the latest version of the code.

There is still much work to improve the README - it doesn't provide
a very good introduction to somebody new to IATI.
requires.io still refers to the old repository name, and
does not work with the new name.
It is useful to have multiple so that the difference between
master and dev can be seen.
@hayfield hayfield added complete A PR that is in a state that is ready for review. and removed incomplete A PR that is in a state that is not ready for review. labels Oct 24, 2017
README.md Outdated

Usage
=====

**WARNING:** This `iati.core` library is currently in active development. **All usage examples are subject to change**, as we iteratively improve functionality. Therefore, the following examples are provided for illustrative purposes only. As the library matures, this message and other documentation will be updated accordingly!
**WARNING:** This `iati.core` library is currently in active development. **All usage examples are subject to change**, as we iteratively improve functionality. Therefore, the following examples are provided for illustrative purposes only. As the library matures, this message and other documentation will be updated accordingly!
Copy link
Contributor

Choose a reason for hiding this comment

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

Exclamation marks are probably overkill lol

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also mentions iati.core

Remove a reference to iati.core
Remove an unnecessary exclamation mark
Copy link
Contributor

@allthatilk allthatilk left a comment

Choose a reason for hiding this comment

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

🐻

@hayfield hayfield merged commit 9bebe6b into dev Oct 24, 2017
@hayfield hayfield deleted the update-readme branch October 24, 2017 15:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
complete A PR that is in a state that is ready for review. docs Changes to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants