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

feat: add check for enterprise github cloud #3069

Merged
merged 4 commits into from
Mar 25, 2024

Conversation

pritamstyz4ever
Copy link
Contributor

Context

We need a mechanism to identify if a user belongs to an enterprise when using the Github Enterprise Cloud configuration for login

Objective

This PR adds configuration to check if the user belongs to the enterprise github cloud

References

screwdriver-cd/scm-github#224

License

I confirm that this contribution is made under a BSD license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@coveralls
Copy link

coveralls commented Mar 25, 2024

Coverage Status

coverage: 95.221% (+0.006%) from 95.215%
when pulling 3eb9de2 on feat/screwdrv-10849-ppaul
into e2cfd14 on master.

Comment on lines 74 to 75
gheCloud: joi.boolean().optional().default(false),
gheCloudSlug: joi.string().optional()
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh yes, on it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated to encapsulate config under scm

tkyi
tkyi previously approved these changes Mar 25, 2024
@pritamstyz4ever pritamstyz4ever merged commit 114b910 into master Mar 25, 2024
2 checks passed
@pritamstyz4ever pritamstyz4ever deleted the feat/screwdrv-10849-ppaul branch March 25, 2024 22:21
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.

None yet

3 participants