Skip to content

Infodev doc review #944

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

Merged
merged 10 commits into from
Oct 26, 2022
Merged

Infodev doc review #944

merged 10 commits into from
Oct 26, 2022

Conversation

aepanchi
Copy link
Contributor

@aepanchi aepanchi commented Oct 18, 2022

Fix errors and mistakes in the documentation.

  • Have you provided a meaningful PR description?

@oleksandr-pavlyk

@coveralls
Copy link
Collaborator

coveralls commented Oct 18, 2022

Coverage Status

Coverage decreased (-0.08%) to 82.058% when pulling 3af9993 on alexandraepan:infodev-doc-review into 59980a2 on IntelPython:master.

@aepanchi
Copy link
Contributor Author

@oleksandr-pavlyk @diptorupd I believe, we can drop asterisks after programming languages.

diptorupd
diptorupd previously approved these changes Oct 26, 2022
Copy link
Contributor

@diptorupd diptorupd left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @alexandraepan

@oleksandr-pavlyk oleksandr-pavlyk dismissed stale reviews from diptorupd and themself via 3af9993 October 26, 2022 13:19
@oleksandr-pavlyk
Copy link
Contributor

Thank you @alexandraepan, I ran pre-commit run --all on your changes and pushed. Once CI is great, I am going to merge.

Copy link
Contributor

@oleksandr-pavlyk oleksandr-pavlyk left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you @alexandraepan

I noticed C++ compilation warnings while compiling DPCTLSyclInterface sources.

I opened #952 with the fix.

@oleksandr-pavlyk oleksandr-pavlyk merged commit 9e934b2 into IntelPython:master Oct 26, 2022
@github-actions
Copy link

Array API standard conformance tests for dpctl=0.14.0dev0=py310h8c27c75_220 ran successfully.
Passed: 33
Failed: 801
Skipped: 3138

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.

4 participants