Skip to content

Move the background jobs to another crate #1661

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

Merged

Conversation

sgrif
Copy link
Contributor

@sgrif sgrif commented Mar 11, 2019

I wrote this code intending to extract it, and I'm planning to release
it after a few more changes which aren't relevant to crates.io. The code
right now is largely the same as what was here before, the main
difference being the use of a different error type.

The error handling will probably churn some more in the near future as I
figure out what I want the error handling story to look like in that
crate

@bors
Copy link
Contributor

bors commented Mar 13, 2019

☔ The latest upstream changes (presumably #1657) made this pull request unmergeable. Please resolve the merge conflicts.

@jtgeibel
Copy link
Member

I merged to resolve the merge conflicts so that I could r+ this, but it looks like the lockfile isn't storing the git SHA and the new build is picking up the latest changes on swirl master.

sgrif added 3 commits March 14, 2019 16:29
I wrote this code intending to extract it, and I'm planning to release
it after a few more changes which aren't relevant to crates.io. The code
right now is largely the same as what was here before, the main
difference being the use of a different error type.

The error handling will probably churn some more in the near future as I
figure out what I want the error handling story to look like in that
crate
@sgrif sgrif force-pushed the sg-move-background-jobs-to-separate-crate branch from a0258b2 to 438ba40 Compare March 14, 2019 22:30
@sgrif
Copy link
Contributor Author

sgrif commented Mar 14, 2019

Locked to a single commit.

@bors r=jtgeibel

@bors
Copy link
Contributor

bors commented Mar 14, 2019

📌 Commit 438ba40 has been approved by jtgeibel

@bors
Copy link
Contributor

bors commented Mar 14, 2019

⌛ Testing commit 438ba40 with merge ef78fb8...

bors added a commit that referenced this pull request Mar 14, 2019
…, r=jtgeibel

Move the background jobs to another crate

I wrote this code intending to extract it, and I'm planning to release
it after a few more changes which aren't relevant to crates.io. The code
right now is largely the same as what was here before, the main
difference being the use of a different error type.

The error handling will probably churn some more in the near future as I
figure out what I want the error handling story to look like in that
crate
@bors
Copy link
Contributor

bors commented Mar 14, 2019

☀️ Test successful - checks-travis
Approved by: jtgeibel
Pushing ef78fb8 to master...

@bors bors merged commit 438ba40 into rust-lang:master Mar 14, 2019
@sgrif sgrif deleted the sg-move-background-jobs-to-separate-crate branch March 18, 2019 18:30
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.

3 participants