Conversation
|
After we land this, we need to set up appveyor as a CI thing, right? Is there anything else we need to do? |
|
Not mandatory; devs can just do it on their forks if you're not keen on it being part of the PR process. I've chosen to use |
|
@willkg lmk if I need to do anything to enable AppVeyor on the repo! |
willkg
left a comment
There was a problem hiding this comment.
This looks good to me! Thank you!
|
@jayvdb Naming the file that is fine by me. I want this to be part of the PR process. I'll figure out whether I can add it on my own and if not, ping @gsnedders for help. Thank you! |
|
Now added. https://ci.appveyor.com/project/gsnedders/html5lib-python/build/1.0.1 ran this as landed. |
|
@gsnedders Thank you! |
This is the first subset of #293 .