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

Use rocm-docs-core for linting #370

Closed
wants to merge 11 commits into from
Closed

Conversation

saadrahim
Copy link
Member

No description provided.

@saadrahim
Copy link
Member Author

@MathiasMagnus I tried using the scripts but there are some issues here.

@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #370 (2df1c5b) into develop (6d3b085) will not change coverage.
Report is 4 commits behind head on develop.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #370   +/-   ##
========================================
  Coverage    58.05%   58.05%           
========================================
  Files           51       51           
  Lines         4815     4815           
========================================
  Hits          2795     2795           
  Misses        2020     2020           
Flag Coverage Δ
gfx906 58.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@Maetveis
Copy link
Contributor

I would prefer to wait for ROCm/rocm-docs-core#489 instead, my reasoning being that GitHub is not the primary platform where development happens for rocRAND, so any issues that this linting would uncover would only come up during the upstreaming pull requests.

@samjwu samjwu added the ci:static-libraries Use this label to run CI with static libraries label Mar 13, 2024
@saadrahim saadrahim requested a review from a team as a code owner March 13, 2024 16:21
@stanleytsang-amd
Copy link
Collaborator

Seems like there's a bunch of failures in CI and also in the files changed section.

@stanleytsang-amd
Copy link
Collaborator

PR is stale. Closing.

Naraenda pushed a commit to StreamHPC/rocRAND that referenced this pull request Oct 28, 2024
…lop_stream'

Resolve "Add small examples to API documentation"

Closes ROCm#370

See merge request amd/libraries/rocRAND!351
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:static-libraries Use this label to run CI with static libraries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants