Update PR template and OWNERS

[Jdubrick]

What does this PR do?:

• Updates the PR template as part of devtools week
• Updates OWNERS file

Which issue(s) this PR fixes:

N/A

PR acceptance criteria:

Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened and linked to this PR, if they are not in the PR scope due to various constraints.

• Unit/Functional tests

• QE Integration test

• Documentation (READMEs, Product Docs, Blogs, Education Modules, etc.)

• Client Impact

• Gosec scans

How to test changes / Special notes to the reviewer:

[thepetk]

/lgtm

from consistency perspective, thinking that we've already migrated from OWNERS to CODEOWNERS for api & registry I'd say we might want to do the same across all repos in the future. WDYT?

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Jdubrick, thepetk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [Jdubrick,thepetk]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Jdubrick]

/lgtm

from consistency perspective, thinking that we've already migrated from OWNERS to CODEOWNERS for api & registry I'd say we might want to do the same across all repos in the future. WDYT?

Yeah I think it would be good to open an issue to look into that. I support it!

[michael-valdron]

/lgtm
from consistency perspective, thinking that we've already migrated from OWNERS to CODEOWNERS for api & registry I'd say we might want to do the same across all repos in the future. WDYT?

Yeah I think it would be good to open an issue to look into that. I support it!

@thepetk @Jdubrick This might work for devfile/library as we only need OWNERS for OpenShift CI, other which use it though i.e. devfile/registry, devfile/registry-operator, and devfile/integration-tests all needed remain with OWNERS for the base directory for this reason.

[thepetk]

/lgtm
from consistency perspective, thinking that we've already migrated from OWNERS to CODEOWNERS for api & registry I'd say we might want to do the same across all repos in the future. WDYT?

Yeah I think it would be good to open an issue to look into that. I support it!

@thepetk @Jdubrick This might work for devfile/library as we only need OWNERS for OpenShift CI, other which use it though i.e. devfile/registry, devfile/registry-operator, and devfile/integration-tests all needed remain with OWNERS for the base directory for this reason.

Yeah right, I had in mind that there was a reason that we've never implemented it in other repos but I've forgotten why. Thanks for the reminder!