Fixes #8596

What changes did you make?

• trim-inactive-members.md added non-PR contribution label
• trim-inactive-members.js added await closePrework()
• same file changed fs.writeFile() to fs.writeFileSync()
• create-new-issue.js returned the issue.html_url and replaced let with const
• wr-schedule-monthly.yml changes
• Includes default permissions from PR Add default permissions to "Schedule Monthly" and "WR Schedule Monthly" workflows #8611

Why did you make the changes (we will use this info to test)?

• Label will prevent check-closed-issue-for-linked-pr.yml from reopening issue
• The await is needed because the function has an await; prevents race condition
• Needed for similar reasons to prevent race condition
• Need to use correct variable
• Simplifies YML and cleans up "Close-New-Issue" by removing extraneous code

CodeQL Alerts

After the PR has been submitted and the resulting GitHub actions/checks have been completed, developers should check the PR for CodeQL alert annotations.

Please let us know that you have checked for CodeQL alerts. Please do not dismiss alerts.

• I have checked this PR for CodeQL alerts and none were found.
• I found CodeQL alert(s), and (select one):
  • I have resolved the CodeQL alert(s) as noted
  • I believe the CodeQL alert(s) is a false positive (Merge Team will evaluate)
  • I have followed the Instructions below, but I am still stuck (Merge Team will evaluate)

Screenshots of Proposed Changes To The Website (if any, please do not include screenshots of code changes)

• No visual changes
• Test logs- schedule-monthly.yml
• Test logs- wr-schedule-monthly.yml

Note to reviewers

If you review this, make sure to see the following resources first:

• Refer to The "Hack for LA's GitHub Actions" Wiki guide needs additional explanation #6537 (comment), especially Tips 6, 7, and 8, if you have not already created you personal environment for testing.
• How to Test GitHub Actions
• Additional Notes for GitHub Actions
• Tokens, Secrets, Scopes, & Permissions

Important note: line numbers and specific code references may have changed slightly since the time of this Wiki- verify with the actual files.

• You will need to have a functioning test environment on your local repo. Refer to Hack for LA's GitHub Actions, especially Tips 6, 7, & 8.
• IMPORTANT: In addition to the 'files changed' in the PR, there are additional changes that you should make to help with testing. If you do not make these edits, you might delete or mis-edit Hack for LA data, delete team members, generate false or junk notifications, and/or create junk issues on HfLA's live Project Board.
• In schedule-monthly.yml, you will need to activate two personal tokens (see Hack for LA's GitHub Actions)
  • HACKFORLA_BOT_PA_TOKEN scopes: admin:org_hook, public_repo
  • HACKFORLA_ADMIN_TOKEN scopes: admin:org_hook, repo, write:org
  • In the same file, around line 12, replace 'hackforla' with your personal repo.
  • Finally, add the line workflow_dispatch: under on: (in line with schedule:)
• In get-contributors-data.js
  • Near line 84 replace with owner: 'hackforla',
  • Near line 85 replace with repo: 'website', (unless your repo is 'website')
• In trim-inactive-members.js:

  • IMPORTANT: Disable the following by replacing:

    await github.request('DELETE /orgs/{org}/teams/{team_slug}/memberships/{username}', {
              org: context.repo.owner,
              team_slug: team,
              username: username,
            });
        }
    

    with:

            console.log('Would be removed: ' + username);
            // await github.request('DELETE /orgs/{org}/teams/{team_slug}/memberships/{username}', {
            //   org: context.repo.owner,
            //   team_slug: team,
            //   username: username,
            // });
          }
    

  • IMPORTANT: Disable the entirety of the closePrework() function, starting around ln 110:

    async function closePrework(member, issueNum){ 
      // Close the assignee's "Pre-work Checklist" and add comment
      // await github.request('PATCH /repos/{owner}/{repo}/issues/{issue_number}', {
      //   owner: org,
      //   repo: repo,
      //   issue_number: issueNum,
      //   state: 'closed'
      // });
      console.log(`Would be closing "Skills Issue" issue number  ${issueNum} for ${member}`);
      // // Add comment to issue
      // await github.request('POST /repos/{owner}/{repo}/issues/{issue_number}/comments', {
      //   owner: org,
      //   repo: repo,
      //   issue_number: issueNum,
      //   body: 'The Hack for LA Bot has closed this issue due to member inactivity.'
      // });
    }
    

• In create-new-issue.js:
  • For const AGENDA_ISSUE_NUM = replace with an existing issue number in your repo
  • The variables for const owner and const repo are ok as is- do not replace.
  • IMPORTANT: replace the following lines exactly as shown:

    let removedList = removeList.map(x => "@ " + x).join("\n");    // important to add space
    let notifiedList = notifyList.map(x => "@ " + x).join("\n");   // important to add space
    

  • Towards the end of the files, comment out: // let milestone = parseInt(issueObject['milestone']);
  • And next: // milestone,
• In /utils/get-team-members.js:
  • Ln 16, replace with org: 'hackforla',
• In /utils/add-team-member.js:
  • Ln 11, replace with org: 'hackforla',
  • Ln 18, replace with org: 'hackforla',
• In /utils/get-timeline.js:
  • Ln 16, replace with owner: 'hackforla',
  • Ln 17, replace with repo: 'website',
• In /utils/post-issue-comment.js:
  • Ln 9, replace with owner: ' <your name> ', NOT 'hackforla'
  • Ln 10, replace with repo: 'website', or however you named your repo