Skip to content
This repository has been archived by the owner on Mar 12, 2024. It is now read-only.

fix: fixing upgrade requirements. #592

Merged
merged 6 commits into from Jan 5, 2023
Merged

fix: fixing upgrade requirements. #592

merged 6 commits into from Jan 5, 2023

Conversation

awais786
Copy link
Contributor

@awais786 awais786 commented Jan 4, 2023

Upgrade job is failing due to pip-tools.
Added few constraints to avoid tests failures.
Upgrade pygithub

https://github.com/openedx/edx-rest-api-client/blob/master/requirements/base.in#L4 is responsible for bringing in the django

@awais786 awais786 marked this pull request as ready for review January 4, 2023 18:44
@@ -129,7 +129,8 @@ def test_get_commits_by_branch_branch_not_found(self):
{
'documentation_url': 'https://developer.github.com/v3/repos/#get-branch',
'message': 'Branch not found'
}
},
headers={}
Copy link
Contributor Author

@awais786 awais786 Jan 4, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@awais786 awais786 merged commit 5869b77 into master Jan 5, 2023
@awais786 awais786 deleted the fixing-upgrade branch January 5, 2023 09:57
mtyaka pushed a commit to mtyaka/tubular that referenced this pull request May 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants