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

remove python version completely #573

Merged
merged 3 commits into from
Apr 25, 2022
Merged

remove python version completely #573

merged 3 commits into from
Apr 25, 2022

Conversation

intrand
Copy link
Contributor

@intrand intrand commented Apr 20, 2022

Description

What's the goal of this PR?

No longer maintain nor offer the legacy python version of reckoner. Any references can be made by looking at a previous release/commit.

What changes did you make?

Removed all of the python stuff :)

What alternative solution should we consider, if any?

We've already decided to drop it

@intrand intrand added the python Pull requests that update Python code label Apr 20, 2022
@intrand intrand self-assigned this Apr 20, 2022
Copy link
Contributor

@lucasreed lucasreed left a comment

Choose a reason for hiding this comment

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

This is a great start, I think there are some python specific things in the docs/ directory too if you wouldn't mind looking through those (mostly around installation I think).

@intrand
Copy link
Contributor Author

intrand commented Apr 20, 2022

This is a great start, I think there are some python specific things in the docs/ directory too if you wouldn't mind looking through those (mostly around installation I think).

Sure thing.

@intrand intrand requested a review from lucasreed April 25, 2022 13:23
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@lucasreed lucasreed left a comment

Choose a reason for hiding this comment

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

Woot! Thanks 🎉

@intrand intrand merged commit ab75433 into master Apr 25, 2022
@intrand intrand deleted the int/no-py branch April 25, 2022 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants