Skip to content

Integration of the CLI within an Azure DevOps Pipeline #5879

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

Open
4 tasks
Tracked by #5414
Adam-it opened this issue Mar 5, 2024 · 4 comments · May be fixed by #6663
Open
4 tasks
Tracked by #5414

Integration of the CLI within an Azure DevOps Pipeline #5879

Adam-it opened this issue Mar 5, 2024 · 4 comments · May be fixed by #6663

Comments

@Adam-it
Copy link
Member

Adam-it commented Mar 5, 2024

🎯 Aim

We have a nice guide on how to use CLI for M365 in GitHub workflows but it's not the only DevOps environment where we may use CLI.
The idea is to add guidance how we may use CLI in Azure DevOps. We should definitely include:

  • some basic intro why would someone want to do that and how to start
  • tips with best practices that might be helpful specifically for script tasks/steps that use CLI. Like for example how you should handle the secret for the login command, or even what type of auth in login command should be used in a pipeline (no the password one but rather go as a app)
  • we should definitely show a working yaml pipeline flow with a basic example that builds and deploys a SPFx package to a app catalog (similar like in the GitHub guide)
  • we should mention the spfx project azuredevops pipeline add command which is currently a open PR so we could wait for it to be merged before we start working on this guide

👍 Additionally

In the GitHub workflow setup guide, we have 'Creating a basic workflow which authenticates with an Microsoft 365 tenant' section which explains how to create a basic workflow using CLI to deploy a SPFx package.
https://pnp.github.io/cli-microsoft365/user-guide/github-actions#creating-a-basic-workflow-which-authenticates-with-an-microsoft-365-tenant

Actually, now we have a command that will generate that for the user 🙂. Along the way we could include this info in the guide as well 👍

@Adam-it Adam-it added docs needs peer review Needs second pair of eyes to review the spec or PR labels Mar 5, 2024
@Adam-it
Copy link
Member Author

Adam-it commented Mar 5, 2024

@pnp/cli-for-microsoft-365-maintainers anything I missed?

@milanholemans
Copy link
Contributor

Looks good to me 👍

@milanholemans milanholemans added help wanted and removed needs peer review Needs second pair of eyes to review the spec or PR labels Mar 5, 2024
@Adam-it
Copy link
Member Author

Adam-it commented Mar 6, 2024

Looks good to me 👍

Thanks, let's wait a bit and if it will not get any traction I will take it

@Adam-it
Copy link
Member Author

Adam-it commented May 5, 2024

I'll take it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants