-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Removed paver code and scripts specific to unit testing #31180
Conversation
@zubairshakoorarbisoft could you please expand the PR description with a little more details about the reason for this change? |
@bradenmacdonald Its kind of draft PR, currently working on it. The reason is to remove
paver code (only tests specific) is that now we are are running test using
pytest that's why need to remove this useless codebase from this repo.
To get the more context on this task, please have a look into this issue #31168
CC: @iamsobanjaved
…On Fri, 21 Oct 2022 at 11:53 PM, Braden MacDonald ***@***.***> wrote:
@zubairshakoorarbisoft <https://github.com/zubairshakoorarbisoft> could
you please expand the PR description with a little more details about the
reason for this change?
And please remove the template text like "Useful information to include:"
and "Please provide detailed step-by-step instructions for testing this
change." if you are not using it.
—
Reply to this email directly, view it on GitHub
<#31180 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANX4P4VZFJE25D3S2JNMCSLWELRBJANCNFSM6AAAAAARJ5ZM7M>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
The content of this email and attachment(s) are confidential and intended
solely for the recipient(s) specified in the message. If you received this
email by error, please alert the sender by replying to this email and
follow with its deletion, so that we can ensure such an error does not
occur in future. If you are not the intended recipient, you are strictly
prohibited to use, copy, disseminate or store this email or its
attachment(s).
|
64b45c0
to
4faebca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
("compare-branch=", "b", "Branch to compare against, defaults to origin/master"), | ||
], share_with=['coverage']) | ||
@timed | ||
def diff_coverage(options): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is being used currently by JS tests so moved this command to this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall but need a few more changes to be merged:
- Update the script paver_autocomplete to remove unnecessary references or the script altogether.
- Update the PR description with a details description & summary of why these changes are being made.
bb84fb2
to
1c9c2ce
Compare
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
EdX Release Notice: This PR has been deployed to the production environment. |
The contents of this file were deleted by #31180; this commit just deletes the leftover empty file.
The contents of this file were deleted by #31180; this commit just deletes the leftover empty file.
The contents of this file were deleted by openedx#31180; this commit just deletes the leftover empty file.
The contents of this file were deleted by #31180; this commit just deletes the leftover empty file.
The contents of this file were deleted by openedx#31180; this commit just deletes the leftover empty file.
Description
Remove the codebase specific for python unit testing in the paver. We have moved to use pytest directly for python unit testing so now removing all obsolete code.