8000 Allow adding repos to teams with specific permissions by borgstrom · Pull Request #625 · sigmavirus24/github3.py · GitHub
[go: up one dir, main page]

Skip to content

Allow adding repos to teams with specific permissions #625

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
merged 2 commits into from
Jul 25, 2016

Conversation

borgstrom
Copy link
Contributor

@sigmavirus24
Copy link
Owner

Thanks @borgstrom!

@itsmemattchung would you have a chance to re-record the cassette that's failing for @borgstrom?

@borgstrom
Copy link
Contributor Author

@sigmavirus24 I fixed the test locally by simply editing the existing cassette and adding the new permission attr to the body for the PUT request.

@itsmemattchung
Copy link
Contributor

Thanks @borgstrom!

@itsmemattchung would you have a chance to re-record the cassette that's failing for @borgstrom?

Yup - taking a look

@itsmemattchung
Copy link
Contributor

I'm re-recording the cassette, however, a single test is failing. This may be a local issue:

tests/integration/test_orgs_team.py::TestTeam::test_add_member PASSED
tests/integration/test_orgs_team.py::TestTeam::test_add_repository FAILED
tests/integration/test_orgs_team.py::TestTeam::test_can_filter_members_by_role PASSED
tests/integration/test_orgs_team.py::TestTeam::test_delete PASSED
tests/integration/test_orgs_team.py::TestTeam::test_edit PASSED
tests/integration/test_orgs_team.py::TestTeam::test_has_repository PASSED
tests/integration/test_orgs_team.py::TestTeam::test_is_member PASSED
tests/integration/test_orgs_team.py::TestTeam::test_members PASSED
tests/integration/test_orgs_team.py::TestTeam::test_remove_member PASSED
tests/integration/test_orgs_team.py::TestTeam::test_remove_repository PASSED
tests/integration/test_orgs_team.py::TestTeam::test_repositories PASSED

I should have a new cassette recorded soon.

@itsmemattchung
Copy link
Contributor

Thanks @borgstrom . I merged the change and included the new cassette.

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
0