Skip to content
This repository has been archived by the owner on Sep 3, 2024. It is now read-only.

Kno220 add duo account with no complex password policy #51

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

mcpeach9
Copy link

@mcpeach9 mcpeach9 commented Jun 5, 2022

No description provided.

@mcpeach9 mcpeach9 requested a review from a team as a code owner June 5, 2022 09:25
Copy link

@chjlarson chjlarson left a comment

Choose a reason for hiding this comment

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

I think it’s great! I don't see any obvious syntax errors that would keep it from executing. Let me know if you're okay with the comment and I can approve!

jupiterone/questions/questions.yaml Outdated Show resolved Hide resolved
chjlarson
chjlarson previously approved these changes Jun 7, 2022
Copy link

@chjlarson chjlarson left a comment

Choose a reason for hiding this comment

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

Looks good to me! Good Job!

ericanagle1
ericanagle1 previously approved these changes Jun 22, 2022
@ericanagle1 ericanagle1 requested a review from chjlarson June 22, 2022 15:54
Copy link
Contributor

@ericanagle1 ericanagle1 left a comment

Choose a reason for hiding this comment

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

Updated bad to BAD

@ericanagle1 ericanagle1 self-requested a review June 22, 2022 20:21
@ericanagle1
Copy link
Contributor

@austinkelleher i updated the gitleaks.yaml so i think we just need a codeowner to approve and this should get pushed, can you confirm?

Copy link

@chjlarson chjlarson left a comment

Choose a reason for hiding this comment

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

Looks good to me! I dont see any obvious syntax errors that would keep this from running! Good work!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants