You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.
This is a very nice script! These are just some improvements to make it better.
General Comments
t would be better to capture the status code of a command or bash function with $?. This will let you early exit and provide an error message rather than attempting to go to the next step of the installation.
E.g. metis-sideload
It might make sense to chain the each value of $? and check if it is equal to 0 before calling the next function.
Specific Comments
metis-sideload
It would be nice if we could edit the partition again after quitting the specified tool in case the user wants to makes further edits, otherwise the user has to start up metis-sideload again and run through the steps again.
post_install.sh
When adding the user in addingUser(), there should be input validation loops for the username and password. Currently the function will fail if given an incorrect username syntax or failing to input the password correctly 2 times. Once it fails, the script will go to the next step, sudoAccess().
final.sh
In the pointsToRemember() function:
pointToRemember() {
sleep 2s
clear
echo"After Reboting into newly installed metis linux system, you must select xinitrc in login screen."echo"There's are options like Dwm, shell and xinitrc above the username field. Strictly select xinitrc, then enter your username and password and login."echo""
sleep 15s
}
Reboting, should be rebooting.
The text was updated successfully, but these errors were encountered:
cause all the tasks inside these functions are being checked and if something wrong goes inside it, it'll exit with status code 1.
Example: If you see checkingConnection;
checkingConnection() {
sleep 2s
clear
echo "Checking internet connection..."
ping -c 3 metislinux.org || networkError
echo "Cool! Internet connection available. Getting ready for installation..."
sleep 3s
}
If ping exits with non zero status code; it'll run networkError function which is
networkError() {
sleep 2s
clear
echo -e "Could not connect to the internet.\nCheck your internet connection and please try again.\nExiting...\nReport any bugs or errors to https://github.com/metis-os/reports/issues"
sleep 2s
exit 1
}
And It'll exit the script with certain error message.
If you think using the status code and giving error after each function make the script better we can do that as well. Other than that you do have good, valid improvment points.
It will take us some time to make changes cause we're working on the disc encryption (many people were demanding it) so, if you do have some time and want to contribute; why don't you open a PR?
Thank you for your response! What's the process for contributing? I only have been coding for work and personal projects and I haven't done any open source contributions before. I'm assuming I have to fork the repo instead of cloning and creating a branch?
I also saw your post about leaving github for codeberg. Will the metis repos be migrated on that platform. If so, I can create an account on codeberg for contributions 😄
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
This is a very nice script! These are just some improvements to make it better.
General Comments
t would be better to capture the status code of a command or bash function with
$?
. This will let you early exit and provide an error message rather than attempting to go to the next step of the installation.E.g.
metis-sideload
It might make sense to chain the each value of
$?
and check if it is equal to0
before calling the next function.Specific Comments
metis-sideload
It would be nice if we could edit the partition again after quitting the specified tool in case the user wants to makes further edits, otherwise the user has to start up metis-sideload again and run through the steps again.
post_install.sh
When adding the user in
addingUser()
, there should be input validation loops for the username and password. Currently the function will fail if given an incorrect username syntax or failing to input the password correctly 2 times. Once it fails, the script will go to the next step,sudoAccess()
.final.sh
In the
pointsToRemember()
function:Reboting
, should berebooting
.The text was updated successfully, but these errors were encountered: