Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Added license headers for pre-unlink/post-link scripts#1695

Merged
oleksandr-pavlyk merged 1 commit intomasterIntelPython/dpctl:masterfrom
add-license-to-dpctl-installation-scriptsIntelPython/dpctl:add-license-to-dpctl-installation-scriptsCopy head branch name to clipboard
May 23, 2024
Merged

Added license headers for pre-unlink/post-link scripts#1695
oleksandr-pavlyk merged 1 commit intomasterIntelPython/dpctl:masterfrom
add-license-to-dpctl-installation-scriptsIntelPython/dpctl:add-license-to-dpctl-installation-scriptsCopy head branch name to clipboard

Conversation

@oleksandr-pavlyk
Copy link
Contributor

Added license headers for pre-unlink/post-link scripts.

These shell scripts are included in the shipped conda package, and require a license header.

This change is to be backported to maintenance/0.17.x once merged to the development branch.

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • Have you checked performance impact of proposed changes?
  • If this PR is a work in progress, are you opening the PR as a draft?

These shell scripts are included in the shipped conda package,
and require a license header.
@github-actions
Copy link

github-actions bot commented May 23, 2024

Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞

@github-actions
Copy link

Array API standard conformance tests for dpctl=0.18.0dev0=py310h15de555_21 ran successfully.
Passed: 890
Failed: 15
Skipped: 91

@coveralls
Copy link
Collaborator

Coverage Status

coverage: 87.915%. remained the same
when pulling 965b9c7 on add-license-to-dpctl-installation-scripts
into 4a06eb1 on master.

Copy link
Contributor

@AndresGuzman-Ballen AndresGuzman-Ballen 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 to me! Thanks!!!

@oleksandr-pavlyk oleksandr-pavlyk merged commit 60fc3eb into master May 23, 2024
@oleksandr-pavlyk oleksandr-pavlyk deleted the add-license-to-dpctl-installation-scripts branch May 23, 2024 14:42
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.