removed aws-sdk/types from not-allowed list

[lukebrdn]

Resolves #30

[github-actions]

Coverage after merging allow-aws-sdk/types into main will be

59.47%	0.00%

Coverage Report

File	Stmts	Branches	Funcs	Lines	Uncovered Lines
src/check-imports
index.ts	0%	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.ts	100%	100%	100%	100%
packages-not-allowed.ts	100%	100%	100%	100%
src/coverage-reporter
cli.ts	0%	0%	0%	0%	10, 12–13, 16, 18–19, 21–22, 25, 25–26, 29, 38, 42, 44, 46, 8–9
comment.ts	0%	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.ts	0%	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.ts	0%	0%	0%	0%	12, 17, 17, 17, 17–18, 21, 29, 29–30, 37, 37, 37–38, 38, 38–39, 8
html.ts	100%	100%	100%	100%
index.ts	0%	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.ts	83.12%	73.68%	75%	87.04%	117, 119–120, 120, 120, 120–122, 124, 126, 134, 83
tabulate.ts	100%	100%	100%	100%
util.ts	100%	100%	100%	100%
src/nocks
github.test.ts	100%	100%	100%	100%
src/publish-beta
compile.ts	0%	100%	0%	0%	10–13, 3–5, 7, 9
files.ts	90.91%	85.71%	100%	90.91%	28–30
github.ts	86.36%	73.68%	100%	90.91%	26, 32, 34–36, 38, 53–55
index.ts	0%	100%	0%	0%	10–11, 13–15, 17–21, 24, 26, 28, 3, 33, 35, 4–5, 7–9
package.ts	92.50%	66.67%	100%	93.94%	22–23, 41
publish.ts	72.22%	100%	50%	75%	23–26

[ramaghanta]

should we remove the other aws-sdk dependencies too?

[adcreare]

I don't think we can because aws-nock doesn't support some of the changes that sdk > 3.193 introduces.
I think we need to fix those before we push this higher.

[ramaghanta]

@lukebrdn missing label. Also adding @adcreare who's the best person to look at this PR

[lukebrdn]

@lukebrdn missing label. Also adding @adcreare who's the best person to look at this PR

I was curious about that. I don't believe past PRs included a label. Not to say we shouldn't start.

[ramaghanta]

@lukebrdn missing label. Also adding @adcreare who's the best person to look at this PR

I was curious about that. I don't believe past PRs included a label. Not to say we shouldn't start.

sorry, my bad. there is no publishing for this repository. So, no need for a label.

[adcreare]

LGTM

[ramaghanta]

lgtm