Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Factor isNotFromAnExternalOrGTMUser out of githubEventHelpers #533

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

chadwhitacre
Copy link
Member

@chadwhitacre chadwhitacre commented Jul 21, 2023

Part of #482, after #532.

I want to see if I can put the GraphQL wrappers on GitHubOrg. Gotta get this helper out of the way first.

modifyDueByDate,
modifyProjectIssueField,
shouldSkip,
} from '@utils/githubEventHelpers';
import { isFromABot } from '@utils/isFromABot';
import { isNotFromAnExternalOrGTMUser } from '@utils/isNotFromAnExternalOrGTMUser';
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note parallel with isFromABot on the previous line.

Copy link
Member

@hubertdeng123 hubertdeng123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, but maybe should change name of PR from isNotFromBlahBlah to isNotFromAnExternalOrGTMUser?

@chadwhitacre chadwhitacre force-pushed the cwlw/factor-out-org branch 2 times, most recently from aa44c03 to 17e4d89 Compare July 21, 2023 21:12
Base automatically changed from cwlw/factor-out-org to main July 21, 2023 21:14
@chadwhitacre chadwhitacre force-pushed the cwlw/isnotfromablahblah branch from 95b1369 to 55ab3d8 Compare July 21, 2023 22:05
@chadwhitacre
Copy link
Member Author

maybe should change name of PR from isNotFromBlahBlah to isNotFromAnExternalOrGTMUser

Boring. :P

@chadwhitacre chadwhitacre changed the title Factor isNotFromBlahBlah out of githubEventHelpers Factor isNotFromAnExternalOrGTMUser out of githubEventHelpers Jul 21, 2023
@chadwhitacre chadwhitacre enabled auto-merge (squash) July 21, 2023 22:06
@chadwhitacre chadwhitacre merged commit 0ad098e into main Jul 21, 2023
@codecov
Copy link

codecov bot commented Jul 21, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01 🎉

Comparison is base (ccc5cca) 84.61% compared to head (55ab3d8) 84.63%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #533      +/-   ##
==========================================
+ Coverage   84.61%   84.63%   +0.01%     
==========================================
  Files          98       99       +1     
  Lines        2503     2505       +2     
  Branches      476      476              
==========================================
+ Hits         2118     2120       +2     
  Misses        378      378              
  Partials        7        7              
Impacted Files Coverage Δ
src/utils/githubEventHelpers.ts 88.52% <ø> (-0.71%) ⬇️
src/brain/issueLabelHandler/followups.ts 100.00% <100.00%> (ø)
src/brain/issueLabelHandler/route.ts 97.36% <100.00%> (+0.03%) ⬆️
src/brain/issueLabelHandler/triage.ts 97.77% <100.00%> (ø)
src/utils/isNotFromAnExternalOrGTMUser.ts 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@chadwhitacre chadwhitacre deleted the cwlw/isnotfromablahblah branch July 21, 2023 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants