Skip to content

Update PR test setup#42

Merged
rkansal47 merged 15 commits intomasterfrom
nsmith--patch-1
Feb 26, 2025
Merged

Update PR test setup#42
rkansal47 merged 15 commits intomasterfrom
nsmith--patch-1

Conversation

@nsmith-
Copy link
Copy Markdown
Owner

@nsmith- nsmith- commented Feb 14, 2025

No description provided.

@nsmith- nsmith- changed the title Update PR test conda setup Update PR test setup Feb 14, 2025
@nsmith-
Copy link
Copy Markdown
Owner Author

nsmith- commented Feb 25, 2025

pre-commit.ci autofix

@nsmith-
Copy link
Copy Markdown
Owner Author

nsmith- commented Feb 26, 2025

@rkansal47 can you approve and merge this?

@rkansal47
Copy link
Copy Markdown
Collaborator

Oh wow nice! Do we not want the Root 6.22 test anymore? Most of our analyses are still using that.

@nsmith-
Copy link
Copy Markdown
Owner Author

nsmith- commented Feb 26, 2025

yeah we should, problem is it is so old we don't have long before all python versions are deprecated for it and removed from conda-forge https://anaconda.org/conda-forge/root/files?version=6.22.8

@rkansal47
Copy link
Copy Markdown
Collaborator

Awesome, LGTM! Damn didn’t realize Python 3.9 is getting deprecated this year…

@rkansal47 rkansal47 self-requested a review February 26, 2025 03:27
@rkansal47 rkansal47 merged commit 6313432 into master Feb 26, 2025
@rkansal47 rkansal47 deleted the nsmith--patch-1 branch February 26, 2025 03:27
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.

2 participants