-
Notifications
You must be signed in to change notification settings - Fork 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
Pip module not there. Introduce a Fallback #1
Comments
Also I used |
We can either kill the setup script if Alternatively, we can switch to a zipapp archive. But implementing cryptographic functions ourselves is not a feasible option. |
@mrigankpawagi There does exist a pre-push git hook which would be much better than pre-commit since, in the case of CherryPicking, it might get skipped. (just like it did for me). Lets try that too |
That is a good point, but it will probably be quite complicated. (Let us say we start with no active changes. Otherwise, you can stash them.) You will have to unroll all the unpushed commits and keep stacking them on the stash. Then in the reverse order, you will have to encrypt the assignment files in each commit, and re-create those commits with the encrypted files instead of the original file. After re-creating the latest commit, you will replace the local assignment files with their encrypted versions. I might be wrong, and would be happy to discuss more. |
@mrigankpawagi checkout #1 What do you think? |
Hi, looks at this
Well, this is a bug. I am quite surprised my
python3
does not have pip. But I personally dont like the fact you are installingcryptography
.So suggestions
requests
library and some good API(find one), encrypt it.--- If the permission is not given, tell
falling back to Manually ciphering
.Notice my assignment (go to my fork, not linking here), I pushed it after I did the assi, yet NOTHING happened, well cause crypto failed. Fallback would be great! ik its easily breakable, but better than nothing.
also introduce like more
exceptions
. Ill put a PR with a possible fallback after discussion.The text was updated successfully, but these errors were encountered: