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

Migrate to issue forms #127

Closed

Conversation

TheEvilSkeleton
Copy link

GitHub announced issue forms a while ago. Issue forms are essentially forms you have to fill out, as opposed to the current Markdown section. It is much easier to use issue forms, looks prettier and some sections may even hard require for user input.

If you want to see it in action, don't hesitate to file issues! https://github.com/TheEvilSkeleton/open-gpu-kernel-modules/issues/new/choose

@CLAassistant
Copy link

CLAassistant commented May 12, 2022

CLA assistant check
All committers have signed the CLA.

@mtijanic mtijanic added the process Workflow-related item label May 16, 2022
@niv niv self-assigned this May 17, 2022
@PAR2020
Copy link
Contributor

PAR2020 commented May 18, 2022

Thanks @TheEvilSkeleton for the suggestion and PR.
@niv has worked up a solution that meets our needs and it should roll out shortly.

@niv niv closed this in b48f850 May 19, 2022
@niv
Copy link
Member

niv commented May 19, 2022

Here we go. :) Thanks for the nudge!

Daasin added a commit to Daasin/OPU-Kernel-Modules that referenced this pull request Nov 23, 2022
* moved array sanity check to before the array access

* Fixed some typos

* [~] Fix typos in NVDEC Engine Descriptor

* Fixed capitalization

* Create CONTRIBUTING.md

* use github forms for issue flow management, fixes NVIDIA#127

* issue template: add automatic tags

* Update HTTP link to HTTPS

http://www.nvidia.com redirects to https://www.nvidia.com/, so linking to http is reduntant.
More importantly, it could be considered a security issue, do to the nature of plain http. An attacker could perform a MITM attack and redirect the HTTP request to a malicious website.
```
> GET / HTTP/1.1
> Host: www.nvidia.com

Response:
< HTTP/1.1 307 Temporary Redirect
< Location: https://www.nvidia.com/
```

* Fix shellcheck errors in conftest

* Remove non-posix local usage from conftest

* Remove trailing whitespace from conftest

* 515.48.07

* 515.57

* update CONTRIBUTING.md to explain cosmetics policy

Also adds a blurb about the missing code style guide.

* nv-pci: Fix nullptr dereference if device was not found

Closes NVIDIA#57

* dp_configcaps: Avoid querying bus if both bLinkActive and bLinkReady is NULL

Doesn't fix a crash -- but avoids redundant work due to a copy paste error.

Closes NVIDIA#56

* osapi: Fix dereferencing NULL pNumaMemSize in rm_get_gpu_numa_info

Copy paste error.

Closes: NVIDIA#55

* dp_connectorimpl: Remove duplicate dscBitsPerPixelIncrement

All other values of the BITS_PER_PIXEL_INCREMENT are handled here.

This one was previously using a bitwise OR in, then also assigning directly after.

Closes: NVIDIA#54

* dp_connectorimpl: Use switch-case for dscBitsPerPixelIncrement

This is an enum, so we can just use a switch case instead of chaning IFs constantly.

* 515.65.01

* 515.76

* 520.61.05

* 520.56.06

* 525.53

Co-authored-by: astolfo <[email protected]>
Co-authored-by: Maneet Singh <[email protected]>
Co-authored-by: Filip Fedoryszyn <[email protected]>
Co-authored-by: TrickyDmitriy <[email protected]>
Co-authored-by: keroeslux <[email protected]>
Co-authored-by: Andy Ritger <[email protected]>
Co-authored-by: Bernhard Stoeckner <[email protected]>
Co-authored-by: Alcaparra <[email protected]>
Co-authored-by: nitepone <[email protected]>
Co-authored-by: Joshua Ashton <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
process Workflow-related item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants