Skip to content

Update AzF profile to be useful #1341

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Sep 14, 2019
Merged

Conversation

rjmholt
Copy link
Contributor

@rjmholt rjmholt commented Sep 13, 2019

PR Summary

I've run the profile collector on the AzF profile (with some difficulty) and collected a new profile that captures the default Az installation.

I've also removed cmdlets that don't work in AzF despite being present, like Start-Job and Enter-PSSession.

I had to make some small changes to the profile collector to not crash (plus one where I realised that collecting a function if we're not able to load it is unhelpful).

PR Checklist

Copy link
Collaborator

@bergmeister bergmeister left a comment

Choose a reason for hiding this comment

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

Looks ok to me but it broke 1 CI test:

    Context Targeting new-form Az profiles alongside older profiles
      [-] Finds AzF problems with a script 7.37s
        KeyNotFoundException: The given key 'InDisconnectedSession' was not present in the dictionary.
        at <ScriptBlock>, C:\projects\psscriptanalyzer\Tests\Rules\UseCompatibleCommands.Tests.ps1: line 334

@rjmholt
Copy link
Contributor Author

rjmholt commented Sep 13, 2019

Impressive that testing picked that up! I'll have to make some further edits

@rjmholt
Copy link
Contributor Author

rjmholt commented Sep 14, 2019

Impressive that testing picked that up!

Have to credit you entirely for that @bergmeister -- you asked for those tests.

Looks like they're working now. Hopefully the Lazys don't hide other faults. I took out parameters and commands manually, and it looks like I took a parameter out without its alias there.

@rjmholt rjmholt merged commit 5e207aa into PowerShell:master Sep 14, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants