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

Add Appveyor and Travis CI (if needed) #46

Closed
jeking3 opened this issue Sep 14, 2017 · 4 comments · Fixed by #49
Closed

Add Appveyor and Travis CI (if needed) #46

jeking3 opened this issue Sep 14, 2017 · 4 comments · Fixed by #49

Comments

@jeking3
Copy link
Contributor

jeking3 commented Sep 14, 2017

We need a suite of tests that exercise the large number of compile-time branches in the project and ensure compatibility with Visual Studio versions (2010 through 2017) and older MinGW 32-bit and newer MinGW-x64.

This was referenced Sep 15, 2017
@jeking3
Copy link
Contributor Author

jeking3 commented Sep 16, 2017

@Lastique Please set milestone to v1.66.0 for this issue.

@jeking3
Copy link
Contributor Author

jeking3 commented Sep 19, 2017

@Lastique I just noticed that my company email was used as the commit author instead of my apache.org email. It should be "jking@apache.org". Is there any way to fix that in the history, or is it impossible now that it was merged?

@Lastique
Copy link
Member

It is possible to rewrite history on develop, but that will cause inconsistencies with the superproject (i.e. some commits in the superproject will be referring to a non-existent commit in Boost.WinAPI). Unless you really want to change that email (e.g. it might get you in trouble with your employer or something) I would prefer to avoid doing this. Please, let me know if you still want me to do this.

@jeking3
Copy link
Contributor Author

jeking3 commented Sep 21, 2017 via email

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

Successfully merging a pull request may close this issue.

2 participants