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

Remove 'warning' from docs tool output #2031

merged 3 commits into from Apr 21, 2021


Copy link

@dsherry dsherry commented Mar 24, 2021

Noticed while doing the 0.21.0 release formatting.

@dsherry dsherry marked this pull request as ready for review March 24, 2021 20:42
Copy link

codecov bot commented Mar 24, 2021

Codecov Report

Merging #2031 (96732ea) into main (070e01e) will decrease coverage by 0.1%.
The diff coverage is n/a.

❗ Current head 96732ea differs from pull request most recent head f6edce5. Consider uploading reports for the commit f6edce5 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##             main    #2031     +/-   ##
- Coverage   100.0%   100.0%   -0.0%     
  Files         293      291      -2     
  Lines       24009    23710    -299     
- Hits        23999    23700    -299     
  Misses         10       10             
Impacted Files Coverage Δ
evalml/automl/ 100.0% <0.0%> (ø)
evalml/data_checks/ 100.0% <0.0%> (ø)
evalml/pipelines/ 100.0% <0.0%> (ø)
evalml/pipelines/ 100.0% <0.0%> (ø)
evalml/model_understanding/ 100.0% <0.0%> (ø)
evalml/tests/automl_tests/ 100.0% <0.0%> (ø)
evalml/data_checks/ 100.0% <0.0%> (ø)
evalml/data_checks/ 100.0% <0.0%> (ø)
evalml/tests/pipeline_tests/ 100.0% <0.0%> (ø)
evalml/tests/data_checks_tests/ 100.0% <0.0%> (ø)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 070e01e...f6edce5. Read the comment docs.

Copy link

@bchen1116 bchen1116 left a comment

Choose a reason for hiding this comment

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

Looks good! Do you have an example of what warning there was when creating the release notes?

Copy link

CLAassistant commented Mar 26, 2021

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor Author

dsherry commented Apr 20, 2021

Looks good! Do you have an example of what warning there was when creating the release notes?

@bchen1116 yep, its a sphinx thing. Search the release notes for ".. warning::" and you'll see them in there.

@dsherry dsherry mentioned this pull request Apr 20, 2021
@dsherry dsherry merged commit 66cb6a0 into main Apr 21, 2021
@dsherry dsherry deleted the ds_format_release_notes branch April 21, 2021 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

4 participants