Skip to content
This repository has been archived by the owner on Sep 25, 2023. It is now read-only.

Add docs build job #536

Merged
merged 26 commits into from
Feb 15, 2023
Merged

Add docs build job #536

merged 26 commits into from
Feb 15, 2023

Conversation

AyodeAwe
Copy link
Contributor

The PR adds a docs_build process to the PR and Build workflows for this repository. The generated docs are synced to s3 for only the build workflows.

@AyodeAwe AyodeAwe added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jan 19, 2023
@github-actions github-actions bot added the conda label Jan 19, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jan 19, 2023 •
edited
Loading

Codecov Report

Base: 71.67% // Head: 71.67% // No change to project coverage 👍

Coverage data is based on head (1b2e8ab) compared to base (883a037).
Patch has no changes to coverable lines.

Additional details and impacted files 
@@              Coverage Diff              @@
##           branch-23.02     #536   +/-   ##
=============================================
  Coverage         71.67%   71.67%           
=============================================
  Files                51       51           
  Lines              2860     2860           
=============================================
  Hits               2050     2050           
  Misses              810      810

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@AyodeAwe AyodeAwe force-pushed the add_docs_build branch 2 times, most recently from bdcbf37 to 0d0c22a Compare January 19, 2023 13:04
@AyodeAwe AyodeAwe marked this pull request as ready for review January 25, 2023 16:00
@AyodeAwe AyodeAwe requested a review from a team as a code owner January 25, 2023 16:00
ajschmidt8
ajschmidt8 reviewed Jan 25, 2023
View reviewed changes
ci/build_docs.sh Outdated Show resolved Hide resolved
ci/build_docs.sh Outdated Show resolved Hide resolved
ci/build_docs.sh Show resolved Hide resolved
@ajschmidt8 ajschmidt8 changed the title add docs_build step Add docs build job Jan 25, 2023
@AyodeAwe
Copy link
Contributor Author 

Warning, treated as error:
/opt/conda/envs/docs/lib/python3.9/site-packages/cusignal/convolution/convolve.py:docstring of cusignal.convolution.convolve.convolve2d:8:Unexpected indentation.

@AyodeAwe
Copy link
Contributor Author

I have reached out to the cusignal team on this error.

@awthomp awthomp mentioned this pull request Jan 31, 2023
Closed
@AyodeAwe AyodeAwe requested a review from a team as a code owner January 31, 2023 18:18
@github-actions github-actions bot removed the Python label Feb 1, 2023
awthomp
awthomp previously approved these changes Feb 1, 2023
View reviewed changes
ajschmidt8
ajschmidt8 reviewed Feb 1, 2023
View reviewed changes
dependencies.yaml Outdated Show resolved Hide resolved
conda/environments/all_cuda-118_arch-x86_64.yaml Outdated Show resolved Hide resolved
ajschmidt8
ajschmidt8 reviewed Feb 1, 2023
View reviewed changes
ci/build_docs.sh Outdated Show resolved Hide resolved
@AyodeAwe AyodeAwe changed the base branch from branch-23.02 to branch-23.04 February 2, 2023 14:28
@AyodeAwe AyodeAwe dismissed awthomp’s stale review February 2, 2023 14:28

The base branch was changed.

ajschmidt8
ajschmidt8 reviewed Feb 6, 2023
View reviewed changes
dependencies.yaml Outdated Show resolved Hide resolved
dependencies.yaml Outdated Show resolved Hide resolved
ajschmidt8
ajschmidt8 reviewed Feb 13, 2023
View reviewed changes
.github/workflows/build.yaml Outdated Show resolved Hide resolved
.github/workflows/pr.yaml Outdated Show resolved Hide resolved
@ajschmidt8 ajschmidt8 mentioned this pull request Feb 14, 2023
Open
@AyodeAwe
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit ab63007 into rapidsai:branch-23.04 Feb 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@awthomp awthomp awthomp approved these changes

@ajschmidt8 ajschmidt8 ajschmidt8 approved these changes

Assignees
No one assigned
Labels
conda improvement Improvement / enhancement to an existing function non-breaking Non-breaking change Python
Projects
cusignal
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AyodeAwe @codecov-commenter @awthomp @ajschmidt8