Skip to content
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

Feature document #185

Merged
merged 57 commits into from
Apr 9, 2021
Merged

Feature document #185

merged 57 commits into from
Apr 9, 2021

Conversation

vaaaaanquish
Copy link
Contributor

@vaaaaanquish vaaaaanquish commented Apr 3, 2021

Update document for v1.0

TODO

  • Fix Warnings
  • Add Japanese documents
  • Update English documents

@@ -84,7 +81,7 @@
# Add any paths that contain custom static files (such as style sheets) here,
# relative to this directory. They are copied after the builtin static files,
# so a file named "default.css" will overwrite the builtin "default.css".
html_static_path = ['_static']
Copy link
Contributor Author

Choose a reason for hiding this comment

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

WARNING because this directory does not exist.

@@ -22,6 +22,7 @@ Table of Contents
task_information
task_settings
slack_notification
using_task_cache_collision_lock
Copy link
Contributor Author

Choose a reason for hiding this comment

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

WARNING because not index using_task_cache_collision_lock.rst

@@ -12,22 +12,23 @@ Prepare following environmental variables:
export SLACK_CHANNEL=channel-name // not "#channel-name", just "channel-name"


A Slack bot token can obtain from `here <https://api.slack.com/apps>`_.
A Slack bot token can obtain from `slack app document <https://api.slack.com/apps>`_.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

WARNING because here is duplicated.

@@ -3,7 +3,6 @@

import luigi

import gokart
Copy link
Contributor Author

Choose a reason for hiding this comment

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

not used

import types
from functools import partial
Copy link
Contributor Author

Choose a reason for hiding this comment

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

not used

python:
version: 3.7
install:
- requirements: docs/requirements.txt
Copy link
Contributor Author

Choose a reason for hiding this comment

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

import gokart

luigi.task.Task.requires.__doc__ = gokart.task.TaskOnKart.requires.__doc__
luigi.task.Task.output.__doc__ = gokart.task.TaskOnKart.output.__doc__
Copy link
Contributor Author

Choose a reason for hiding this comment

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

WARNING because undefined label for luigi label.

sphinx-doc/sphinx#6211

Copy link
Collaborator

@hirosassa hirosassa left a comment

Choose a reason for hiding this comment

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

Thanks for your contribute! LGTM

@@ -7,7 +7,7 @@ license = "MIT"
readme = "README.md"
homepage = "https://github.com/m3dev/gokart"
repository = "https://github.com/m3dev/gokart"
documentation = ""
documentation = "https://gokart.readthedocs.io/en/latest/"
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

vaaaaanquish and others added 25 commits April 7, 2021 00:22
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
Co-authored-by: kyoh.maruo <18437866+snowhork@users.noreply.github.com>
Co-authored-by: kyoh.maruo <18437866+snowhork@users.noreply.github.com>
@vaaaaanquish vaaaaanquish mentioned this pull request Apr 8, 2021
7 tasks
@vaaaaanquish vaaaaanquish changed the base branch from master to 1.0.0 April 8, 2021 19:54
vaaaaanquish and others added 2 commits April 9, 2021 12:09
Co-authored-by: hirosassa <hiro.sassa@gmail.com>
@vaaaaanquish vaaaaanquish merged commit b0caad6 into 1.0.0 Apr 9, 2021
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.

2 participants