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.

Sign up for GitHub

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

Jump to bottom

Remove remaining nosetest settings in favor of pytest #8129

Merged
merged 1 commit into from Feb 14, 2023
Merged

Remove remaining nosetest settings in favor of pytest #8129

merged 1 commit into from Feb 14, 2023

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Feb 12, 2023 •
edited by misl6

Remove remaining nosetest settings in favor of pytest.

Maintainer merge checklist

  • Title is descriptive/clear for inclusion in release notes.
  • Applied a Component: xxx label.
  • Applied the api-deprecation or api-break label.
  • Applied the release-highlight label to be highlighted in release notes.
  • Added to the milestone version it was merged into.
  • Unittests are included in PR.
  • Properly documented, including versionadded, versionchanged as needed.

@misl6 misl6 added this to the 2.2.0 milestone Feb 12, 2023
@misl6 misl6 added the Component: tests/CI Tests, CI, GitHub settings label Feb 12, 2023
misl6
misl6 approved these changes Feb 14, 2023
View reviewed changes
Copy link
Member

@misl6 misl6 left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@misl6 misl6 merged commit 5c5cb51 into kivy:master Feb 14, 2023
@cclauss cclauss deleted the remove-nosetest-settings branch February 14, 2023 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@misl6 misl6 misl6 approved these changes

Assignees
No one assigned
Labels
Component: tests/CI Tests, CI, GitHub settings
Projects
Kivy's Control Tower
Status: Done
Milestone
2.2.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@cclauss @misl6