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

Typo combined #9560

Merged
merged 14 commits into from
Dec 19, 2024
Merged

Typo combined #9560

merged 14 commits into from
Dec 19, 2024

Conversation

samuel-marsh
Copy link
Collaborator

Hi Seurat Team,

Given updates to dev branch and CI checks I thought it might be helpful to condense a number of typo fixes into single PR that is on current develop branch.

Thanks to authors of original PRs @josura @agdenadel @tnqvuong @kew24 @nrockweiler @eroell @mauriziopaul @AlbertoFabbri93.

Best,
Sam

@rharao
Copy link
Contributor

rharao commented Dec 19, 2024

I snuck a typo fix into my PR at 05aedba ; you can cherry pick it if you want

@samuel-marsh
Copy link
Collaborator Author

@rharao done. adding couple more too

Copy link
Contributor

@dcollins15 dcollins15 left a comment

Choose a reason for hiding this comment

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

I cannot tell you how much I appreciate this @samuel-marsh! 🙌

LG(reat)TM 🚀

@dcollins15 dcollins15 merged commit 2a70a24 into satijalab:develop Dec 19, 2024
2 checks passed
@rharao
Copy link
Contributor

rharao commented Dec 19, 2024

With this merge, the open PRs are down to 25-- few enough to fit on one page 😄 great momentum you have going @dcollins15

@samuel-marsh
Copy link
Collaborator Author

@dcollins15 always happy to help! :)

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.

3 participants