-
Notifications
You must be signed in to change notification settings - Fork 295
add contact options to README #3201
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
removed gitter from contact list
@bjlittle @pp-mo @kaedonkers I would like to see this merged today, if possible. |
@tv3141 Rather than expecting you to rebase and finish tasks, I have done it myself here. I hope this is ok. Please let me know if you would like me to change this at all. |
README.md
Outdated
|
||
* Report bugs, or suggest new features using an Issue or Pull Request on [Github](https://github.com/SciTools/iris). You can also comment on existing Issues and Pull Requests. | ||
* For discussions from a user perspective you could join our [SciTools User's Google Group](https://groups.google.com/forum/#!forum/scitools-iris). | ||
* For those involved in developing Iris we also have a [Google Group](https://groups.google.com/forum/#!forum/scitools-iris-dev). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you replace the link text "Google Group" with something more specific, like "Iris Developers Group" ?
Otherwise all looks good to me. Positive progress !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pp-mo Will do...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pp-mo I also removed a rogue apostrophe. Although this name change does mean that there is now an 'a' which wants to be an 'an' (L.101). Shall I fix that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll fix it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh no! That's still wrong, hang on, I'm doing a lot of context switching so far today, I'll get this right in a minute...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh no that is right after all. Don't panic, I've obviously just got my Monday brain in.
This is an amended (gitter link removed) and rebased version of #3092 to add some contact details to the Iris README.