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

DS3: Update tests to use bases.py #4138

Merged

Conversation

nicholassaylor
Copy link
Contributor

What is this fixing or adding?

Updates import to bases.py rather than TestBase.py since the latter is deprecated

How was this tested?

Running unit tests

@nicholassaylor
Copy link
Contributor Author

@nex3 Requesting your review

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Nov 3, 2024
@NewSoupVi NewSoupVi added the waiting-on: world-maintainer Issue/PR is waiting for feedback or approval by the maintainer of a world. label Nov 4, 2024
@Exempt-Medic Exempt-Medic added the is: refactor/cleanup Improvements to code/output readability or organizization. label Nov 4, 2024
@nex3
Copy link
Contributor

nex3 commented Nov 5, 2024

LGTM

Copy link
Member

@black-sliver black-sliver left a comment

Choose a reason for hiding this comment

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

Thanks

@black-sliver black-sliver merged commit 930529e into ArchipelagoMW:main Nov 9, 2024
18 checks passed
@nicholassaylor nicholassaylor deleted the darksouls-test-base branch November 9, 2024 18:30
AustinSumigray pushed a commit to AustinSumigray/Archipelago that referenced this pull request Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: refactor/cleanup Improvements to code/output readability or organizization. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. waiting-on: world-maintainer Issue/PR is waiting for feedback or approval by the maintainer of a world.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants