Skip to content

Conversation

@adcreare
Copy link
Contributor

@adcreare adcreare commented Aug 15, 2023

Closes #59
Simple setup until we have an "action" to check for failed "actions"

@adcreare adcreare added the MINOR label Aug 15, 2023
@github-actions
Copy link

Coverage after merging check-published-catch-errors into main will be

54.55%▾ -1.01%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src/check-imports
   index.ts0%0%0%0%10–11, 13, 15–17, 17–19, 21–22, 22–23, 3, 34, 36, 38, 4, 43, 45, 6–7, 9
   package-lock-file-util.ts100%100%100%100%
   packages-not-allowed.ts100%100%100%100%
src/check-label
   check-label.ts92.98%77.78%100%95.56%62–63, 71–72
   index.ts0%100%0%0%11, 16, 18, 3, 5, 7, 9
src/check-pr-reviews
   index.ts0%0%0%0%10–12, 14–15, 15, 17, 17, 17–18, 22–23, 26, 28, 28, 3, 30, 30, 30, 36, 4, 40–41, 44, 5, 50, 52, 54, 59, 61, 7–8
src/check-published
   index.ts0%0%0%0%10, 12, 14, 20–22, 26–27, 3, 30–31, 33–34, 36–39, 4, 42, 46–47, 49, 5, 50–51, 54, 56, 56–59, 6, 63, 65, 67, 7, 72, 74, 8
   slack.ts0%100%0%0%20–24, 3, 30–31, 35–36, 4, 45, 48–49, 5, 58, 7
src/comment-npm-publish
   index.ts0%100%0%0%10–11, 13–15, 19, 22, 24, 26, 3, 31, 33, 4–5, 7, 9
src/coverage-reporter
   cli.ts0%0%0%0%10, 12–13, 16, 18–19, 21–22, 25, 25–26, 29, 38, 42, 44, 46, 8–9
   comment.ts0%0%0%0%10, 12, 14–16, 16–17, 17, 23, 23, 29–30, 30–31, 34–37, 37, 37–38, 38, 38, 38, 38, 40–41, 41–42, 42, 48, 48, 8
   delete-old-comments.ts0%0%0%0%13, 20–23, 25, 34–36, 36–37, 41, 43, 43, 43, 43, 43, 49, 54–57, 59, 67, 8
   get-changes.ts0%0%0%0%12, 17, 17, 17, 17–18, 21, 29, 29–30, 37, 37, 37–38, 38, 38–39, 8
   html.ts100%100%100%100%
   index.ts0%0%0%0%100–101, 11–12, 14–17, 19, 21, 24–26, 26, 26–27, 27, 27–31, 33–34, 34, 36–37, 40, 40, 40–41, 41, 41, 41, 43, 46, 52, 52, 52–57, 57–60, 63–64, 66, 66–67, 70–72, 74, 74–75, 78, 78, 78–79, 8, 86, 86–87, 9, 98
   lcov.ts83.12%73.68%75%87.04%125, 127–128, 128, 128, 128–130, 132, 134, 143, 83
   tabulate.ts100%100%100%100%
   util.ts100%100%100%100%
src/github-api
   index.ts74.68%58.97%100%78.18%120–122, 139–140, 142, 164–166, 171–173, 187–189, 194–196, 207–209, 221–222, 35, 41, 43–45, 47, 56–58, 63–65, 81–82, 95–97
src/nocks
   github.test.ts86.84%100%80%88.89%131, 137, 32
src/prepare-beta
   index.ts0%100%0%0%10–13, 16, 18, 20, 25, 27, 3–5, 7, 9
   package.ts92%0%100%95.45%21–22
src/publish-beta
   compile.ts0%100%0%0%10–13, 3–5, 7, 9
   files.ts91.18%87.50%100%90.91%28–30
   index.ts0%100%0%0%10–11, 13–15, 17–21, 27, 29, 3, 31, 36, 38, 4–5, 7–9
   package.ts93.94%50%100%96.43%22–23
   publish.ts72.22%100%50%75%23–26

@adcreare adcreare requested a review from carlansley August 15, 2023 11:28
@adcreare adcreare merged commit 41e4290 into main Aug 17, 2023
@adcreare adcreare deleted the check-published-catch-errors branch August 17, 2023 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check-Published handle exception and still notify to slack

3 participants