Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add AppVeyor testing#359

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
willkg merged 2 commits intohtml5lib:masterfromjayvdb:add-appveyor
Nov 9, 2017
Merged

Add AppVeyor testing#359

willkg merged 2 commits intohtml5lib:masterfromjayvdb:add-appveyor
Nov 9, 2017

Conversation

jayvdb
Copy link
Contributor

This is the first subset of#293 .

@jayvdb
Copy link
ContributorAuthor

@willkg
Copy link
Contributor

After we land this, we need to set up appveyor as a CI thing, right? Is there anything else we need to do?

@jayvdb
Copy link
ContributorAuthor

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.appveyor.yml instead of the defaultappveyor.yml , which means that there is one additional setup step : specifying the file in the AppVeyor settings.
I'm having to use the default filename if you'd prefer.

@willkgwillkg added this to the1.0 milestoneNov 8, 2017
@gsnedders
Copy link
Member

@willkg lmk if I need to do anything to enable AppVeyor on the repo!

Copy link
Contributor

@willkgwillkg left a comment

Choose a reason for hiding this comment

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

This looks good to me! Thank you!

@willkgwillkg merged commit85bc5fa intohtml5lib:masterNov 9, 2017
@willkg
Copy link
Contributor

@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!

@gsnedders
Copy link
Member

gsnedders commentedNov 9, 2017
edited
Loading

@willkg
Copy link
Contributor

@gsnedders Thank you!

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@willkgwillkgwillkg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.0
Development

Successfully merging this pull request may close these issues.

3 participants
@jayvdb@willkg@gsnedders

[8]ページ先頭

©2009-2025 Movatter.jp