Skip to content

Comments

Enable simple TravisCI testing#202

Merged
MartinNowak merged 1 commit intodlang:masterfrom
wilzbach:travis
Dec 17, 2016
Merged

Enable simple TravisCI testing#202
MartinNowak merged 1 commit intodlang:masterfrom
wilzbach:travis

Conversation

@wilzbach
Copy link
Contributor

As seen with #200 it's probably not a bad idea to enable CI for this repo ;-)

This adds a very simple bash script that tests basic install / uninstall features (by simply installing them and checking the output of -version).
I didn't add a check for the latest version (default command) as this obviously changes.

A non-D environment on Travis was selected on purpose, of course.

Would someone with admin rights be so kind to enable this project for Travis?
(it passes on my fork)

@wilzbach
Copy link
Contributor Author

Ping @MartinNowak @CyberShadow

Copy link
Member

@MartinNowak MartinNowak left a comment

Choose a reason for hiding this comment

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

LGTM as a start

["dmd-2016-10-19"]="DMD64 D Compiler v2.073.0-master-878b882"
["dmd-master-2016-10-24"]="DMD64 D Compiler v2.073.0-master-ab9d712"
["ldc-1.0.0"]="LDC - the LLVM D compiler (1.0.0):"
["gdc-4.9.3"]="gdc (crosstool-NG crosstool-ng-1.20.0-232-gc746732 - 20150825-2.066.1-58ec4c13ec) 4.9.3"
Copy link
Member

Choose a reason for hiding this comment

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

Should test name resolutions as well. The values could be globs matched via ~= or regexes for grep/egrep.

@MartinNowak MartinNowak merged commit 3edd39c into dlang:master Dec 17, 2016
@wilzbach wilzbach deleted the travis branch December 17, 2016 22:43
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