-
Notifications
You must be signed in to change notification settings - Fork 26
Fix some errors in API docs and link to DTMO #239
Conversation
Codecov Report
@@ Coverage Diff @@
## master #239 +/- ##
=======================================
Coverage 80.05% 80.05%
=======================================
Files 11 11
Lines 1053 1053
=======================================
Hits 843 843
Misses 210 210
Continue to review full report at Codecov.
|
d22bb8a
to
5e21878
Compare
README.md
Outdated
Spark bindings for Mozilla Telemetry | ||
PySpark bindings for Mozilla Telemetry | ||
|
||
## Usage |
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 think less is more for this section. Let's just do something like:
## Documentation
Online documentation is available at https://mozilla.github.io/python_moztelemetry/
That way if the dtmo link ever changes, we only need to update it in one place.
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.
will do
docs/conf.py
Outdated
@@ -55,17 +55,17 @@ | |||
|
|||
# General information about the project. | |||
project = u'python_moztelemetry' | |||
copyright = u'2016, Mozilla Firefox Data Platform' | |||
copyright = u'2016-2018, Mozilla Firefox Data Platform' |
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 we just leave out the year? That piece of information always seems to go out of date
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.
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.
r+ with comments addressed
No description provided.