-
Notifications
You must be signed in to change notification settings - Fork 653
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
DOCS-#3936: Update copyright year #3937
DOCS-#3936: Update copyright year #3937
Conversation
Signed-off-by: Igoshev, Yaroslav <yaroslav.igoshev@intel.com>
aa2c5e6
to
4ff1113
Compare
Codecov Report
@@ Coverage Diff @@
## master #3937 +/- ##
===========================================
- Coverage 85.53% 36.00% -49.54%
===========================================
Files 198 185 -13
Lines 16540 15860 -680
===========================================
- Hits 14148 5710 -8438
- Misses 2392 10150 +7758
Continue to review full report at Codecov.
|
@@ -48,7 +48,7 @@ def noop_decorator(*args, **kwargs): | |||
export_config_help(configs_file_path) | |||
|
|||
project = "Modin" | |||
copyright = "2018-2021, Modin" | |||
copyright = "2018-2022, Modin" |
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.
is there any reason this is "Modin" and not "Modin Developers" or vice versa?
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.
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 it is weird that we have Modin Developers
in the NOTICE
file, but not here, we can change it in a new PR.
@jeffreykennethli do you want to open a new issue for 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.
opened #3962
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.
@YarShev LGTM, thanks!
@@ -48,7 +48,7 @@ def noop_decorator(*args, **kwargs): | |||
export_config_help(configs_file_path) | |||
|
|||
project = "Modin" | |||
copyright = "2018-2021, Modin" | |||
copyright = "2018-2022, Modin" |
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 it is weird that we have Modin Developers
in the NOTICE
file, but not here, we can change it in a new PR.
@jeffreykennethli do you want to open a new issue for it?
Signed-off-by: Igoshev, Yaroslav <yaroslav.igoshev@intel.com>
Signed-off-by: Igoshev, Yaroslav yaroslav.igoshev@intel.com
What do these changes do?
flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
git commit -s
docs/developer/architecture.rst
is up-to-date