Skip to content

Commit 7249d85

Browse files
committed
Logs
1 parent bd6c7e9 commit 7249d85

File tree

3 files changed

+10
-2
lines changed

3 files changed

+10
-2
lines changed

README.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,3 +56,11 @@ Then you should be good to go.
5656
### Security
5757
5858
We force the use of [`pull_request_target`](https://github.blog/2020-08-03-github-actions-improvements-for-fork-and-pull-request-workflows/) as a workflow event to ensure that someone cannot change the CODEOWNER files at the same time as having that change be used to validate if they can merge.
59+
60+
### Dev
61+
62+
Use `npx jest --watch` to run tests.
63+
64+
### Deploy
65+
66+
Use the GH UI to make a tag and release

index.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ async function commentOnMergablePRs() {
3939
core.info(`Changed files: \n\n - ${changedFiles.join("\n - ")}`)
4040

4141
const codeowners = findCodeOwnersForChangedFiles(changedFiles, cwd)
42-
core.info(`Code-owners: \n\n - ${changedFiles.join("\n - ")}`)
42+
core.info(`Code-owners: \n\n - ${codeowners.join("\n - ")}`)
4343

4444
if (!codeowners.length) {
4545
console.log("This PR does not have any code-owners")
@@ -128,6 +128,7 @@ function getFilesNotOwnedByCodeOwner(owner, files, cwd) {
128128
function findCodeOwnersForChangedFiles(changedFiles, cwd) {
129129
const owners = new Set()
130130
const codeowners = new Codeowners(cwd);
131+
console.log(cwd, codeowners.codeownersFilePath)
131132

132133
for (const file of changedFiles) {
133134
const filesOwners = codeowners.getOwner(file);

test-code-owners-repo

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)