Skip to content

Update github lists #163

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Dec 11, 2017
Merged

Update github lists #163

merged 1 commit into from
Dec 11, 2017

Conversation

SimonSapin
Copy link
Contributor

This is the result of running:

cargo run --release create-lists
cp work/shared/lists/gh-* .

In particular, this includes servo/servo which is now listed in language:rust searches. CC #133

This is the result of running:

```
cargo run --release create-lists
cp work/shared/lists/gh-* .
```

In particular, this includes servo/servo which is now listed
in `language:rust` searches.

CC rust-lang#133
@SimonSapin SimonSapin changed the title Update github lists. Update github lists Dec 7, 2017
@aidanhs
Copy link
Member

aidanhs commented Dec 8, 2017

Thanks! I'll give it a shot in the requested run for rust-lang/rust#45002 (comment), if you'd want to keep an eye on that.

@SimonSapin
Copy link
Contributor Author

Sounds good.

@SimonSapin
Copy link
Contributor Author

That thread says "run started" but this PR is not merged, did you make this run use a custom branch instead of crater master?

@aidanhs
Copy link
Member

aidanhs commented Dec 8, 2017

I did curl -sSL https://patch-diff.githubusercontent.com/raw/rust-lang-nursery/crater/pull/163.patch | git apply on the box to temporarily apply the changes.

@aidanhs
Copy link
Member

aidanhs commented Dec 11, 2017

More test successes, fewer failures overall...seems good.

@aidanhs aidanhs merged commit 04d6b78 into rust-lang:master Dec 11, 2017
@SimonSapin SimonSapin deleted the app-lists branch December 11, 2017 15:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants