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

Delete tests/screenshots directory #9006

Merged
merged 1 commit into from
Apr 2, 2024
Merged

Delete tests/screenshots directory #9006

merged 1 commit into from
Apr 2, 2024

Conversation

RayBB
Copy link
Collaborator

@RayBB RayBB commented Apr 2, 2024

As far as I can tell, these tests are completely unused.
Found them while looking for docs!

If anyone still uses this please let me know so we can update how to use them and when!

Technical

Testing

Screenshot

Stakeholders

@RayBB RayBB added the Needs: Review This issue/PR needs to be reviewed in order to be closed or merged (see comments). [managed] label Apr 2, 2024
Copy link
Collaborator

@cdrini cdrini left a comment

Choose a reason for hiding this comment

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

Lgtm; following the precedent on #8933

@cdrini cdrini self-assigned this Apr 2, 2024
@cdrini cdrini merged commit 44e250f into master Apr 2, 2024
4 checks passed
@cdrini cdrini deleted the RayBB-patch-3 branch April 2, 2024 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Review This issue/PR needs to be reviewed in order to be closed or merged (see comments). [managed]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants