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

🐧 doc(shell script install) 📜 #16

Merged
merged 3 commits into from
Dec 21, 2022
Merged

Conversation

@adriens adriens changed the title doc(shell script install) 🐧 doc(shell script install) 📜 Dec 21, 2022
@adriens adriens requested a review from Dougniel December 21, 2022 09:16
@adriens adriens added BUILD Toute activité liée à la création de nouvelles fonctionnalités documentation Improvements or additions to documentation quickfix labels Dec 21, 2022
@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

tested on KillerCoda Ubuntu playground

@Dougniel
Copy link
Member

Hello @adriens, ça serait pas plus prudent et concis de ne mettre à disposition le lien de SDKman sans copier leurs doc d'install qui. peut changer ? En plus ça marche uniquement sur du Debian

C'est dommage de parler de Linux alors que ça peut marcher sur tous les OS vu que c'est du Java

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

Hello @adriens, ça serait pas plus prudent et concis de ne mettre à disposition le lien de SDKman sans copier leurs doc d'install qui. peut changer ? En plus ça marche uniquement sur du Debian

👍 En fait, je ferai une version qui dit juste Java 8 prérequis, je déméagerai cette version sur Killercoda et raccourcirai celle du README. En fait, c'est un premier draft pour Killercoda et attaquer la partie des tests d'utilisation chiffrement/déchiffrement.

@Dougniel
Copy link
Member

Dans ce cas là autant le mettre ailleurs que sur le README.md non ?

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

Dans ce cas là autant le mettre ailleurs que sur le README.md non ?

👍 Let's do this 🚀

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

@Dougniel

@Dougniel Dougniel merged commit dec4de1 into develop Dec 21, 2022
@Dougniel Dougniel deleted the shell-script-install branch December 21, 2022 22:18
@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

🚀

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

can you in addition drop me two exec/encryp decrypt examples so i'll complete the tutorial with them ?

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

(at least for Poem file)

@Dougniel
Copy link
Member

There are on src/tests/resources

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

can you please drop me some command lines ?

@Dougniel
Copy link
Member

The command help is not clear enough ? Give me you feedback and I will try yo improve it

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

In fact, I only need to get the password you used to generate poem-with-password.7z

image

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

image

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

image

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

git clone git@github.com:opt-nc/j7zip.git
cd j7zip/src/test/resources/

# Compress & protect with password
j7zip --password=mysecret a poem.7z poem.txt
file poem.7z
mv poem.txt poem.txt.org

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

... now I want i back 😅

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

image

@adriens
Copy link
Member Author

adriens commented Dec 21, 2022

🆘 @Dougniel to get the file back and readable

@Dougniel
Copy link
Member

image

There is a bug on parameters verifications, I opened an issue

The reason of this error is that a destination path is needed (but is not well documented)

@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.0.1-develop.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@adriens adriens mentioned this pull request Dec 23, 2022
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUILD Toute activité liée à la création de nouvelles fonctionnalités documentation Improvements or additions to documentation quickfix released on @develop released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants