Skip to content
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

AppVeyor did not push new version #75

Closed
ferventcoder opened this issue Apr 19, 2017 · 10 comments
Closed

AppVeyor did not push new version #75

ferventcoder opened this issue Apr 19, 2017 · 10 comments

Comments

@ferventcoder
Copy link
Member

No description provided.

@ferventcoder
Copy link
Member Author

@lawrencegripper

To combat this I wanted to see what was going on and added the notifications for appveyor to chocolatey-build-status at googlegroups dot com

@javydekoning
Copy link
Contributor

AppVeyor never started the deploy.ps1 script.

@lawrencegripper
Copy link
Contributor

Looks like the repo moving location has left appveyor a bit confused. I don't have access to the github repo settings anymore so I can't check and update the webhook stuff, will take a look now at appveyor to see if there is anything I can update that side.

Attempted to add you to appveyor but need an email address, if you want to share one privately dm on twitter @lawrencegripper

@lawrencegripper
Copy link
Contributor

Sweet, looks like updating the appveyor build to point to the new repo location has done the trick. I've manually triggered a build for now, as it has missed the webhooks for the commits to master, and the build completed for v2.3.1.96. Output here: https://ci.appveyor.com/project/LawrenceGripper/cchoco/branch/master

Would be worth validating if the hook settings in github need updating too or if it was just ignoring them because they came from a repo it didn't think it was building for.

@ferventcoder checked your blog for email address, looks like your contact page 500's

@lawrencegripper
Copy link
Contributor

lawrencegripper commented Apr 19, 2017

Looks like the repo move meant the appveyor didn't detect this as a build of master branch, thought it was from a fork. Created a PR to resolve this here: #76 Once the PR is merged we should see a deploy to powershell gallery.

On the plus side it let me test the webhooks and appear to be triggering correctly so was just the appveyor changes required.

@ferventcoder
Copy link
Member Author

@lawrencegripper ferventcoder at gmail

@ferventcoder
Copy link
Member Author

@lawrencegripper you are now an admin on this repo.

@lawrencegripper
Copy link
Contributor

@ferventcoder now added to the appveyor with all the permissions, first time I've ever granted permissions through appveyor so apologies if I've got it wrong.

Also build pushed updated module :)

image

@ebekker
Copy link
Contributor

ebekker commented Apr 19, 2017

It looks like the 4th component of the version is now set to '0' (started with the prior build), but previously it was the AV build number (I'm guessing?). Was that change by design?

@ferventcoder
Copy link
Member Author

@ebekker no idea - just coming into this.

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

No branches or pull requests

4 participants