-
-
Notifications
You must be signed in to change notification settings - Fork 14
Adding basic modules including python and cython code + setup file + etc. #1
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
base: master
Are you sure you want to change the base?
Conversation
Anyone objections on merging this branch in, to include an example of how to handle Cython code? |
Should we close this PR? It seems that all the changes it incorporates are already present in the @mkoeppe 's branch. |
I am currently at Sage Days 117. There was a demo earlier this week on how to share code and create a package where we used sage_sample. I am now taking a look at all the PR that have been here for years now... No discussion has never been made about who should merge PRs. The current PR has conflicts. I let the author or someone else fix them. |
I think we should get rid of sage_sample and instead point people to real projects that follow best practices. |
I second Matthias. All this is very outdated. |
Ok, personally, I don't know because I did not follow the evolution of the practices since many years. Do you have a list of |
Yeah, the current version of I still believe that there is value in having an easy to find minimal example that people It could be a cookie cutter instead. Cheers, |
No description provided.