Skip to content

/ PSScriptAnalyzer

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

Fix Ubuntu Appveyor build #1320

Merged
merged 6 commits into from Aug 27, 2019
Merged

Fix Ubuntu Appveyor build #1320

merged 6 commits into from Aug 27, 2019

Conversation

@bergmeister
Copy link
Collaborator

bergmeister commented Aug 22, 2019 •
edited

PR Summary

A minimalized and ready to be merged version of PR #1293 Thanks to @JamesWTruher for the hard work of finding the issue/solution. Tests get now also correctly registered (probably something that Appveyor fixed)

PR Checklist
bergmeister added 5 commits Aug 22, 2019
@bergmeister
try fix appveyor ubuntu build
1491d4c
@bergmeister
temporarily enable only ubuntu
4c360ad
@bergmeister
remove redundant linux customisation
225da89
@bergmeister
fix compatibility analyser build
9a5c636
@bergmeister
final tweak to make it ready for PR
Loading status checks…
8224843
@bergmeister bergmeister requested review from rjmholt and JamesWTruher Aug 22, 2019
@rjmholt
rjmholt reviewed Aug 23, 2019
View changes
tools/appveyor.psm1 Outdated Show resolved Hide resolved
@rjmholt
rjmholt approved these changes Aug 23, 2019
View changes
@bergmeister @rjmholt
Update tools/appveyor.psm1
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
59d1e6f 
Co-Authored-By: Robert Holt <rjmholt@gmail.com>
@JamesWTruher
JamesWTruher approved these changes Aug 27, 2019
View changes
@JamesWTruher JamesWTruher merged commit 5cd3065 into PowerShell:master Aug 27, 2019
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
license/cla All CLA requirements met.
Details
@bergmeister bergmeister mentioned this pull request Aug 27, 2019
Open
4 of 6 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rjmholt rjmholt

@JamesWTruher JamesWTruher

Assignees
No one assigned
Labels
Area - Build
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@bergmeister @rjmholt @JamesWTruher
You can’t perform that action at this time.