Skip to content

/ docs

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

Jump to bottom

PowerShell Quickstart #284

Open
wants to merge 9 commits into
base: main
from
Open

PowerShell Quickstart #284

wants to merge 9 commits into from
+214 −0

Conversation

@potatoqualitee
Copy link

@potatoqualitee potatoqualitee commented Oct 8, 2020 •
edited

Related issue: #283

Why:

As a PowerShell developer, I find GitHub Actions extremely useful and very straightforward. PowerShell is built into each of the runner images, and is supported out of the box. It has a strong CI/CD presence and it makes a ton of sense to include PowerShell in the quickstart.

Note that I used the Building and testing Python as the template for language and format for consistency.

Also, two other PRs must be merged for this PR to be complete:

actions/cache#431
actions/starter-workflows#661

What's being changed:

Added an entire PowerShell / Pester quickstart

Check off the following:

Not sure what reviewed my changes in staging means, but I did look at it in my repo. Perhaps a link can be added to the issue template?
@potatoqualitee
Create building-and-testing-powershell.md
14e9343
@potatoqualitee
add versions
db0cdbd 
guessing version support, based on python
@potatoqualitee
remove . github example, add 'bin'
d7900f3
@potatoqualitee
remove errant brace
712bc2e
@potatoqualitee
no multi-line, pipe not needed
4b381ec
@potatoqualitee
add consistency and shells
8a661b5
@potatoqualitee
fix formatting and code
1038367
@potatoqualitee
added recommendation
Loading status checks…
92012f1
@welcome
Copy link

@welcome welcome bot commented Oct 8, 2020

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.
@deploy-to-heroku deploy-to-heroku bot temporarily deployed to docs-284--powershell Oct 8, 2020 Inactive
@github-actions github-actions bot added this to Triage in Docs team reviews Oct 8, 2020
@potatoqualitee potatoqualitee mentioned this pull request Oct 8, 2020
Open
@potatoqualitee
add to index
Loading status checks…
6caa306
@janiceilene
Copy link
Contributor

@janiceilene janiceilene commented Oct 8, 2020

Thanks @potatoqualitee 💝 We'll get this over to the @github/docs-content-ecosystem to review!
@janiceilene janiceilene moved this from Triage to Ecosystem in Docs team reviews Oct 8, 2020
@potatoqualitee potatoqualitee changed the title Powershell Quickstart PowerShell Quickstart Oct 8, 2020
@potatoqualitee
Copy link
Author

@potatoqualitee potatoqualitee commented Oct 9, 2020

Note that I recommended a PSScriptAnalyzer Action in the comments. The Pester project just reached out and suggested we recommend their gorgeous action as well, I'd love to!

https://github.com/marketplace/actions/publish-unit-test-results

Is this something I can throw into the comments or expand upon? I didn't know about their action before and now I'm going to update my own projects.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
actions ecosystem
Projects
Docs team reviews
Ecosystem
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@potatoqualitee @janiceilene
You can’t perform that action at this time.