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

Initial work to get ado token command working. #228

Merged
merged 7 commits into from
Mar 16, 2023

Conversation

kyle-rader-msft
Copy link
Contributor

This adds a functioning version of the ado token command.

There is refactoring cleanup work that will come after this to extract the common logic into a more testable flow that's re-usable by the ado pat and aad commands.

@kyle-rader-msft kyle-rader-msft requested a review from a team as a code owner March 10, 2023 20:29
reillysiemens
reillysiemens previously approved these changes Mar 10, 2023
mvanchaa
mvanchaa previously approved these changes Mar 10, 2023
@kyle-rader-msft kyle-rader-msft self-assigned this Mar 15, 2023
@kyle-rader-msft kyle-rader-msft added the enhancement New feature or request label Mar 15, 2023
Copy link
Contributor

@shalinikhare27 shalinikhare27 left a comment

Choose a reason for hiding this comment

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

There are bunch of stylecop warnings that I see in every PR from src/AdoPat/PatOptions.cs. Can we fix them in a follow up PR as it's distracting.

src/AzureAuth/Commands/Ado/CommandToken.cs Outdated Show resolved Hide resolved
src/AzureAuth/Commands/Ado/CommandToken.cs Show resolved Hide resolved
src/AzureAuth/Commands/Ado/CommandToken.cs Show resolved Hide resolved
src/AzureAuth/Commands/Ado/CommandToken.cs Show resolved Hide resolved
src/AzureAuth/Commands/Ado/CommandToken.cs Show resolved Hide resolved
src/AzureAuth/Commands/Ado/CommandToken.cs Show resolved Hide resolved
@kyle-rader-msft
Copy link
Contributor Author

There are bunch of stylecop warnings that I see in every PR from src/AdoPat/PatOptions.cs. Can we fix them in a follow up PR as it's distracting.

I've opened a PR to migrate to Sonar Lint, and can then work on cleaning up it's warnings. StyleCop inherently isn't going to work because it's not updating for modern versions of C#.

@kyle-rader-msft kyle-rader-msft merged commit 93f01a4 into main Mar 16, 2023
@kyle-rader-msft kyle-rader-msft deleted the user/kyrader/token2 branch March 16, 2023 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants