Repository: bokeh/bokeh

Issue:
[DOCS] Update links to bokehplots.com, gitter and mailing-list
Dev documentation still points to https://bokehplots.com/pages/contact.html.  bokehplots.com has out-dated information and is being deprecated.  On the gitter @bryevdv said he want's those links to point to bokeh.org instead.


Discussion:
@bryevdv , there are a number of links out to bokehplots.com from the release notes in `sphinx/source/docs/releases/` and on blog.bokeh.org.   What are your thoughts on updating those links?  

It's probably right to leave them alone.  I think most people understand that the release notes make sense at the time of release but things get updated as newer releases are made.  However, I could see how you might not want people to hit 404's if bokehplots.com eventually gets taken down.

If there is a reasonable new link to update to, my ideal preference is definitely that they be updated. That said, I don't think it's a high priority so if it's a lot of work, or excessively tedious, leaving them alone is also fine. 
There aren't that many.  I think most of them are links to the "roadmap" and I see https://bokeh.org/roadmap/ that looks like the right place to link.

After taking the contacts.rst out, I was curious whether Sphinx checked for broken refs in the documentation.  Looks like there is a configuration called 'nitpicky' that can warn about those. https://www.sphinx-doc.org/en/master/usage/configuration.html?highlight=nitpicky#confval-nitpicky

Do you guys have any experience with trying to turn on nitpicky?  I'm building the docs right now with it to see if it generates a bunch of garbage or points me at places where contacts needs to be updated.
The issue with checking links during the build is that it makes the build take longer but more importantly, sometimes the build gets broken due to things outside your control (e.g. we used to check GH issue links, but occasionally GH is down and when that happens the build broke for no good reason). I would very much like to start performing link checks on our site but I think I'd rather do it outside the context of builds (also because there are sites besides the docs that could be checked)
Makes sense.  Also looks like sphinx nitpicky gets confused about what's available in the autogenerated refguide stuff:
`Warning, treated as error:
/home/ubuntu/environment/sphinx/source/docs/dev_guide/models.rst:41:py:class reference target not found: bokeh.core.properties.Property`
@bryevdv , has the bokeh-dev mailing list mentioned on the [bindings page](https://bokeh.pydata.org/en/latest/docs/dev_guide/bindings.html) moved onto the discourse as well?

Bokeh dev mailing list was shuttered a long time ago. The nearest thing now is the Development category on the Discourse:

https://discourse.bokeh.org/c/development

Resolving patch:
diff --git a/scripts/deps.py b/scripts/deps.py
--- a/scripts/deps.py
+++ b/scripts/deps.py
@@ -15,8 +15,18 @@ def _setup(**kw): data.update(kw)
     return data
 
 meta_src = jinja2.Template(open("conda.recipe/meta.yaml").read())
-meta_src = yaml.load(meta_src.render(load_setup_py_data=load_setup_py_data),
-                     Loader=yaml.FullLoader)
+try:
+    meta_src = yaml.load(meta_src.render(load_setup_py_data=load_setup_py_data),
+                         Loader=yaml.FullLoader)
+except AttributeError as e:
+    # Loader=yaml.FullLoader added in pyyaml 5.1 because of:
+    # https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation
+    # isn't available on conda for python=3.5
+    # fall back to calling without loader if it isn't available
+    if 'FullLoader' in repr(e):
+        meta_src = yaml.load(meta_src.render(load_setup_py_data=load_setup_py_data))
+    else:
+        raise
 
 section = {
     "build"  : meta_src["requirements"]["build"],
diff --git a/sphinx/source/conf.py b/sphinx/source/conf.py
--- a/sphinx/source/conf.py
+++ b/sphinx/source/conf.py
@@ -169,14 +169,14 @@
         ('Github', '//github.com/bokeh/bokeh'),
     ),
     'ABOUT': (
-        ('Vision and Work', 'vision'),
-        ('Team',            'team'),
-        ('Citation',        'citation'),
-        ('Contact',         'contact'),
+        ('Roadmap', 'https://bokeh.org/roadmap'),
+        ('Team',            'https://bokeh.org/team'),
+        ('Citation',        'https://bokeh.org/citation'),
+        ('Contact',         'https://bokeh.org'),
     ),
     'SOCIAL': (
         ('Contribute', 'contribute'),
-        ('Mailing list', '//groups.google.com/a/anaconda.com/forum/#!forum/bokeh'),
+        ('Discourse', 'https://discourse.bokeh.org'),
         ('Github', '//github.com/bokeh/bokeh'),
         ('Twitter', '//twitter.com/BokehPlots'),
     ),


Test patch:
diff --git a/sphinx/source/docs/dev_guide/testing.rst b/sphinx/source/docs/dev_guide/testing.rst
--- a/sphinx/source/docs/dev_guide/testing.rst
+++ b/sphinx/source/docs/dev_guide/testing.rst
@@ -339,7 +339,7 @@ resources:
 
   to your commit message.
 
-.. _contact the developers: https://bokehplots.com/pages/contact.html
+.. _contact the developers: https://bokeh.org
 .. _custom markers: http://pytest.org/latest/example/markers.html#working-with-custom-markers
 .. _pytest: https://docs.pytest.org
 .. _selenium webdriver: http://docs.seleniumhq.org/docs/03_webdriver.jsp