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 configuration #5

Merged
merged 1 commit into from
Dec 6, 2017
Merged

Conversation

vlovo
Copy link
Contributor

@vlovo vlovo commented Dec 5, 2017

intial commit of appveyor configuration

@coin-au-carre coin-au-carre merged commit 32838e6 into coin-au-carre:master Dec 6, 2017
@coin-au-carre
Copy link
Owner

coin-au-carre commented Dec 6, 2017

Seems like there is a wrong path ? https://ci.appveyor.com/project/coin-au-carre/slacking

Btw for the contributor section maybe your email provided is not the one associated with github ?
See https://code.bradymower.com/why-arent-my-commits-showing-up-on-my-contributions-graph-52141efa7542

@vlovo
Copy link
Contributor Author

vlovo commented Dec 6, 2017

Thanks, I fixed the email adress in my TortoiseGit configuration. Effectively there was a typo......

cmake --build . --target check

has to be replace by
cmake --build .

(there is no target check ...)

@coin-au-carre
Copy link
Owner

Ok nice feel free to commit this change so we cant test the new 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 this pull request may close these issues.

2 participants