Skip to content

Conversation

nrc
Copy link
Member

@nrc nrc commented Oct 29, 2018

No description provided.

@Centril
Copy link
Contributor

Centril commented Oct 30, 2018

Let's try to rerun this and see if CI is happier...

@anp
Copy link
Contributor

anp commented Nov 1, 2018

@nrc can you rebase your branch onto anp/master and push? CI should pass once you do.

@nrc
Copy link
Member Author

nrc commented Nov 2, 2018

Rebased!

@anp
Copy link
Contributor

anp commented Nov 2, 2018

Hm, it seems that the user database we have from github doesn't have one of the added usernames in it:

thread 'teams::test::team_members_exist' panicked at 'unable to verify team member from database.
if you're running this for tests, make sure you've pulled github users from prod: DieselError(NotFound)', libcore/result.rs:983:5

These usernames are case sensitive, so I think you may need to capitalize eh2406's username, maybe others?

EDIT: I should also note that when you rebased you pulled in an updated user list, so that error message is now a bit misleading.

@nrc
Copy link
Member Author

nrc commented Nov 4, 2018

OK, trying again...

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