Skip to content

Refactor appveyor.psm1 to ci.psm1#8733

Merged
TravisEz13 merged 7 commits into
PowerShell:masterfrom
RDIL:patch-3
Jan 25, 2019
Merged

Refactor appveyor.psm1 to ci.psm1#8733
TravisEz13 merged 7 commits into
PowerShell:masterfrom
RDIL:patch-3

Conversation

@RDIL
Copy link
Copy Markdown
Contributor

@RDIL RDIL commented Jan 24, 2019 •
edited
Loading

PR Summary

AppVeyor removed, so file has been renamed, as well as many AppVeyor references / env vars / etc. have been removed.

PR Context

PR Checklist

TravisEz13
TravisEz13 requested changes Jan 24, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@TravisEz13 TravisEz13 left a comment

Choose a reason for hiding this comment

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

Thanks a lot.

Just one comment.

Comment thread tools/ci.psm1 Outdated
@TravisEz13
Copy link
Copy Markdown
Member

TravisEz13 commented Jan 24, 2019

This was work we planned. FYI, We also planned on merging travis.ps1 and removing duplicate code (It doesn't need to be done in the same PR).

@TravisEz13 TravisEz13 self-assigned this Jan 24, 2019
PaulHigin
PaulHigin suggested changes Jan 24, 2019
View reviewed changes
Comment thread tools/ci.psm1 Outdated
TravisEz13 and others added 2 commits January 24, 2019 14:39
@TravisEz13 @RDIL
Update tools/ci.psm1
2aaaf0c 
Co-Authored-By: RDIL <jumbocakefdx@gmail.com>
@RDIL
Address feedback
b40661d
@TravisEz13
Copy link
Copy Markdown
Member

TravisEz13 commented Jan 24, 2019

@PaulHigin Please update your review.

@RDIL
Copy link
Copy Markdown
Contributor Author

RDIL commented Jan 25, 2019

@PaulHigin I fixed the problems, please re-review

@RDIL
Copy link
Copy Markdown
Contributor Author

RDIL commented Jan 25, 2019

CLA bot is 🅱️roken

@TravisEz13
Copy link
Copy Markdown
Member

TravisEz13 commented Jan 25, 2019 •
edited
Loading

No worries, The CLA bot previously approved this PR.

@RDIL
Copy link
Copy Markdown
Contributor Author

RDIL commented Jan 25, 2019 •
edited
Loading

Windows build now failing for unknown reason. Looks like a file I haven't touched meaning the failure is not an issue with this PR

@TravisEz13
Copy link
Copy Markdown
Member

TravisEz13 commented Jan 25, 2019

We found a test regressed due to an environmental issue.

@TravisEz13
Copy link
Copy Markdown
Member

TravisEz13 commented Jan 25, 2019

merged the fix for the test. I'm going to try to just retry the windows CI. If it fails again with the same error, ask me to rebase the branch with the fix.

@TravisEz13 TravisEz13 merged commit 04a31ff into PowerShell:master Jan 25, 2019
@TravisEz13
Copy link
Copy Markdown
Member

TravisEz13 commented Jan 25, 2019

Used admin rights to merge. CLA did not run on the last commit, but it approved the commit before that.

@TravisEz13
Copy link
Copy Markdown
Member

TravisEz13 commented Jan 25, 2019

Thanks again @RDIL

@RDIL RDIL deleted the patch-3 branch January 26, 2019 00:03
@iSazonov iSazonov added the CL-Tools Indicates that a PR should be marked as a tools change in the Change Log label Feb 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TravisEz13 TravisEz13 TravisEz13 approved these changes

@adityapatwardhan adityapatwardhan adityapatwardhan approved these changes

@anmenaga anmenaga Awaiting requested review from anmenaga

@daxian-dbw daxian-dbw Awaiting requested review from daxian-dbw

+1 more reviewer

@PaulHigin PaulHigin PaulHigin approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@TravisEz13 TravisEz13

Labels

CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@RDIL @TravisEz13 @adityapatwardhan @PaulHigin @iSazonov