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

Fixed missing comma in the csv dumper. #35029

Merged
merged 1 commit into from
Jul 27, 2016

Conversation

eulerdisk
Copy link
Contributor

@eulerdisk eulerdisk commented Jul 25, 2016

Added missing comma before file_line_end.

r? @nrc

@eulerdisk eulerdisk closed this Jul 25, 2016
@eulerdisk eulerdisk reopened this Jul 25, 2016
@nrc
Copy link
Member

nrc commented Jul 25, 2016

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Jul 25, 2016

📌 Commit f694809 has been approved by nrc

@bors
Copy link
Contributor

bors commented Jul 26, 2016

⌛ Testing commit f694809 with merge 6cb03d6...

@bors
Copy link
Contributor

bors commented Jul 26, 2016

💔 Test failed - auto-win-msvc-64-opt

@alexcrichton
Copy link
Member

@bors: retry

On Tue, Jul 26, 2016 at 4:40 AM, bors notifications@github.com wrote:

💔 Test failed - auto-win-msvc-64-opt
https://buildbot.rust-lang.org/builders/auto-win-msvc-64-opt/builds/5098


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#35029 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95N9mFYoV0yhiAwt1ZdLIY9cg5oCaks5qZfIzgaJpZM4JT91u
.

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 26, 2016
…dumper, r=nrc

Fixed missing comma in the csv dumper.

Added missing comma before `file_line_end`.

r? @nrc
bors added a commit that referenced this pull request Jul 27, 2016
@bors bors merged commit f694809 into rust-lang:master Jul 27, 2016
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