Skip to content
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

Fix github url strings (org edx -> openedx) #384

Merged
merged 3 commits into from
Sep 13, 2022
Merged

Conversation

sarina
Copy link
Contributor

@sarina sarina commented Sep 7, 2022

This PR was autogenerated

This pr replaces the old GitHub organization, github.com/edx, with the new GitHub organization, github.com/openedx.

Tagging @openedx/tcril-engineering for review, but others are welcome to provide review.

Ref: openedx/axim-engineering#42

@codecov
Copy link

codecov bot commented Sep 7, 2022

Codecov Report

Base: 94.40% // Head: 94.40% // No change to project coverage 👍

Coverage data is based on head (3f74365) compared to base (13c66f0).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #384   +/-   ##
=======================================
  Coverage   94.40%   94.40%           
=======================================
  Files          28       28           
  Lines        3004     3004           
  Branches      240      240           
=======================================
  Hits         2836     2836           
  Misses        150      150           
  Partials       18       18           
Flag Coverage Δ
unittests 94.40% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@@ -4,7 +4,7 @@ This readme contains the description of different files in the current directory
* middleware.py: This contains an authentication middleware that adds an authenticated user to the requests that are made while running provider verification.
* views.py: This will have provider state setup view which is hit during provider verification to setup appropriate DB state per interaction
* utils.py: Different DB utilities used by views.py
* video-encode-manager-edx-val.json: The only pact file is a duplicate of pact file from https://github.com/edx/video-encode-manager/blob/a4f4c12db0c3919858df1e2fa58ee74071bc264a/encode_manager/apps/core/tests/pact_tests/pacts/video-encode-manager-edx-val.json. This is temporarily needed until pact broker can be setup. Until then, any update in the original pact should be ported in the json in VAL.
* video-encode-manager-edx-val.json: The only pact file is a duplicate of pact file from https://github.com/openedx/video-encode-manager/blob/a4f4c12db0c3919858df1e2fa58ee74071bc264a/encode_manager/apps/core/tests/pact_tests/pacts/video-encode-manager-edx-val.json. This is temporarily needed until pact broker can be setup. Until then, any update in the original pact should be ported in the json in VAL.
Copy link
Contributor

Choose a reason for hiding this comment

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

video-encode-manager is a private repo in edX Github org. The org name change should not be done here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@DawoudSheraz thanks! I pushed a commit.

@sarina sarina merged commit e541a96 into master Sep 13, 2022
@sarina sarina deleted the tcril/fix-gh-org-url branch September 13, 2022 18:18
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.

2 participants