Skip to content

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

Closed
wants to merge 5 commits into from

Conversation

corinnebosley
Copy link
Member

This is an amended (gitter link removed) and rebased version of #3092 to add some contact details to the Iris README.

@corinnebosley
Copy link
Member Author

@bjlittle @pp-mo @kaedonkers I would like to see this merged today, if possible.

@corinnebosley
Copy link
Member Author

@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).
Copy link
Member

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 !

Copy link
Member Author

Choose a reason for hiding this comment

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

@pp-mo Will do...

Copy link
Member Author

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?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll fix it.

Copy link
Member Author

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...

Copy link
Member Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants