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

Test TOML #1012

Merged
merged 1 commit into from
Jan 24, 2019
Merged

Test TOML #1012

merged 1 commit into from
Jan 24, 2019

Conversation

iamed2
Copy link
Contributor

@iamed2 iamed2 commented Jan 24, 2019


I added a fake uuid for the TOML package, do you think something specific should go there?

Includes #1010

I modified number printing to fix the printing test. @StefanKarpinski does this impact you? I noticed you made the change that printed quoted values as a fallback.

@StefanKarpinski
Copy link
Member

bors try

bors bot added a commit that referenced this pull request Jan 24, 2019
Copy link
Member

@StefanKarpinski StefanKarpinski left a comment

Choose a reason for hiding this comment

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

Looks great to me, let's see how Bors feels about it.

@bors
Copy link
Contributor

bors bot commented Jan 24, 2019

try

Build failed

@iamed2
Copy link
Contributor Author

iamed2 commented Jan 24, 2019

ERROR: failed to clone from https://github.com/JuliaRegistries/General.git, error: GitError(Code:ERROR, Class:OS, failed to send request: The server name or address could not be resolved

seems unrelated

@00vareladavid
Copy link
Contributor

Sometimes the network connection is faulty

bors try

bors bot added a commit that referenced this pull request Jan 24, 2019
@bors
Copy link
Contributor

bors bot commented Jan 24, 2019

.travis.yml Outdated Show resolved Hide resolved
ext/TOML/Project.toml Outdated Show resolved Hide resolved
@fredrikekre
Copy link
Member

bors try

bors bot added a commit that referenced this pull request Jan 24, 2019
@bors
Copy link
Contributor

bors bot commented Jan 24, 2019

@fredrikekre
Copy link
Member

please squash to one commit and then we can merge.

@StefanKarpinski
Copy link
Member

We can also just use the GitHub squash and merge button...

@fredrikekre
Copy link
Member

fredrikekre commented Jan 24, 2019

Yea. Btw, my suggested way to test on multiple julia versions seems to not work...

https://travis-ci.org/JuliaLang/Pkg.jl/jobs/483987312#L422

@iamed2
Copy link
Contributor Author

iamed2 commented Jan 24, 2019

Squashed and (I believe) fixed the multiple Julia version testing

@fredrikekre
Copy link
Member

It's a bit awkward that one must repeat the script part etc for each stage, but seems to be no other way...

bors r+

bors bot added a commit that referenced this pull request Jan 24, 2019
1012: Test TOML r=fredrikekre a=iamed2



Co-authored-by: Eric Davies <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jan 24, 2019

Build failed

.travis.yml Outdated Show resolved Hide resolved
@fredrikekre
Copy link
Member

bors try

bors bot added a commit that referenced this pull request Jan 24, 2019
@fredrikekre
Copy link
Member

Nice, seems to work.

bors r+

bors bot added a commit that referenced this pull request Jan 24, 2019
1012: Test TOML r=fredrikekre a=iamed2



Co-authored-by: Eric Davies <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jan 24, 2019

@bors
Copy link
Contributor

bors bot commented Jan 24, 2019

@bors bors bot merged commit a6cadd5 into JuliaLang:master Jan 24, 2019
@iamed2 iamed2 deleted the ed/test-toml branch January 24, 2019 23:09
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.

4 participants