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

Discontinue SmartSim support for python 3.8 #544

Merged
merged 13 commits into from
Apr 17, 2024

Conversation

AlyssaCote
Copy link
Contributor

Python 3.8 is no longer going to be supported because it's near end of life. I've gone through and removed/updated everything that I could find! I added no-merge because I am not sure if there's a specific order that we need to merge all the repos.

@AlyssaCote AlyssaCote marked this pull request as draft April 10, 2024 16:12
@AlyssaCote AlyssaCote marked this pull request as ready for review April 11, 2024 22:20
Copy link

codecov bot commented Apr 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.85%. Comparing base (1267a9a) to head (5e7c175).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##           develop     #544       +/-   ##
============================================
+ Coverage    69.44%   83.85%   +14.40%     
============================================
  Files           65       65               
  Lines         4553     4553               
============================================
+ Hits          3162     3818      +656     
+ Misses        1391      735      -656     
Files Coverage Δ
smartsim/__init__.py 100.00% <ø> (ø)
smartsim/_core/utils/helpers.py 83.21% <100.00%> (+1.39%) ⬆️

... and 30 files with indirect coverage changes

Copy link
Member

@MattToast MattToast left a comment

Choose a reason for hiding this comment

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

One small request, but nothing worth holding up approval over! LGTM!!

pyproject.toml Outdated Show resolved Hide resolved
Copy link
Contributor

@mellis13 mellis13 left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM

@AlyssaCote AlyssaCote changed the title [no-merge] Drop python 3.8 Drop python 3.8 Apr 17, 2024
@AlyssaCote AlyssaCote merged commit 4c9643c into CrayLabs:develop Apr 17, 2024
36 checks passed
@amandarichardsonn amandarichardsonn added API break Issues that include incompatible API changes short task Issues that can be completed and reviewed quickly type: chore Issue outside of general development labels Apr 25, 2024
@amandarichardsonn amandarichardsonn changed the title Drop python 3.8 Discontinue SmartSim support for python 3.8 Apr 26, 2024
ankona pushed a commit to ankona/SmartSim that referenced this pull request May 6, 2024
Python 3.8 is nearing its end of life so we're no longer supporting it.
[ committed by @AlyssaCote ]
[ reviewed by @MattToast @mellis13 ]
ankona pushed a commit to ankona/SmartSim that referenced this pull request May 7, 2024
Python 3.8 is nearing its end of life so we're no longer supporting it.
[ committed by @AlyssaCote ]
[ reviewed by @MattToast @mellis13 ]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API break Issues that include incompatible API changes short task Issues that can be completed and reviewed quickly type: chore Issue outside of general development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants