ropensci / visdat

Preliminary Exploratory Visualisation of Data
https://docs.ropensci.org/visdat/
Other
450 stars 47 forks source link

Appveyor #80

Closed maelle closed 6 years ago

maelle commented 6 years ago

Hi @njtierney :wave:

It seems your Appveyor hook gets failed deliveries. Right now the builds are under your account, although the badge says otherwise (we're still thinking about where to have Appveyor builds, let it stay under your account for now :pray: sorry about the confusion)

Thanks and again sorry for the confusion as we're currently trying to harmonize and improve our Appveyor strategy 😀

maelle commented 6 years ago

I forgot to mention that their email address is team@appveyor.com

maelle commented 6 years ago

Maybe try that from Appveyor first

" If you believe that you need to re-create webhook, you can find correct Webhook URL on General project settings in AppVeyor UI. Use it when creating a Webhook on GitHub. Select “Push” and “Pull request” events. You can also delete unneeded webhook manually on GitHub. Or you can create new project on AppVeyor and correct webhook will be created automatically.

If repo owner changed, you can may also need to re-authorize GitHub at https://ci.appveyor.com/account."
njtierney commented 6 years ago

Thanks for this, Maëlle! :)

I haven't had a lot of time to look at visdat lately, I'm hoping to make another release in the next 4-6 weeks, I'll try handle this in the next few weeks.

maelle commented 6 years ago

Thanks!

njtierney commented 6 years ago

OK great, looking at this now, I think that things are building properly, I went to the appveyor page and asked it to build again and it refrehsed from the one 10 months ago to the most recent commit from 10 months ago

njtierney commented 6 years ago

Tests currently running 🤞

njtierney commented 6 years ago

OK, tests are good, just updated the README !

njtierney commented 6 years ago

Thanks Maëlle!

maelle commented 6 years ago

I still see the webhook as broken in settings?

njtierney commented 6 years ago

Hmmm, ah, I see this now - sorry I thought that this was more to do with the badge. I actually don't quite understand how this gets failed deliveries, but

If you believe that you need to re-create webhook, you can find correct Webhook URL on General project settings in AppVeyor UI. Use it when creating a Webhook on GitHub. Select “Push” and “Pull request” events. You can also delete unneeded webhook manually on GitHub. Or you can create new project on AppVeyor and correct webhook will be created automatically. If repo owner changed, you can may also need to re-authorize GitHub at https://ci.appveyor.com/account."

I'm not sure how to really handle this, the webhook URL on the settings page in appveyor is the same as the one on the github one in settings.

Maybe I email the team and give them the latest delivery info, as you suggested initially, or, I can delete the appveyor project and start a new one?

maelle commented 6 years ago

Given their impatience with my emails, if you don't care too much about the project history, it's probably easier to delete and re start the Appveyor project 😢

njtierney commented 6 years ago

OK can do :)

njtierney commented 6 years ago

OK so I deleted and created a new project, it's building OK: https://ci.appveyor.com/project/njtierney/visdat

But maybe it takes a while for this to update?

njtierney commented 6 years ago

Current status:

image
njtierney commented 6 years ago

OK so on my machine I get this:

image

But that is not reflected on the github site:

image
maelle commented 6 years ago

I see a green badge 🎉 why are there 2 Appveyor webhooks now? 🙈 both work but one should be enough? Sorry and thanks for your patience.

Reg issue labels, why do you use labels and not milestones for versions btw? ☺

njtierney commented 6 years ago

Yay green lights!

hmmm, not sure - let me look into why there are 2 webhooks.

Good question re labels vs milestones - I'm not sure why I use labels! What is the difference between a label and a milestone? :)

njtierney commented 6 years ago

OK so removed the extra webhook :)

maelle commented 6 years ago

Ok so I think this is solved (and for anyone reading this one day and thinking I'm rude, we discussed milestones elsewhere 😸 ). Thanks for your patience @njtierney!

njtierney commented 6 years ago

OK great! Thanks for helping me sort this out and being so helpful, @maelle :)