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

[web] Update README with live ISO hints #988

Merged
merged 3 commits into from
Jan 19, 2024

Conversation

nagu165
Copy link
Contributor

@nagu165 nagu165 commented Jan 10, 2024

  • Added extra information regarding the download procedure for Agama ISO files.
  • Gave an overview of how to run Agama using the virtual machine

- Added extra information regarding the download procedure for Agama ISO files.
- Gave an overview on how to run Agama using virtual machine
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@ancorgs
Copy link
Member

ancorgs commented Jan 11, 2024

First of all, thanks a lot for the contribution. It's really nice that you want to add some clarifications to avoid newcomers confusion. But...

In its current form, this pull request adds quite some text with three points in which (according to previous comments from me and others):

  • The first point contains a direct link that can lead to errors
  • The second one is redundant with the previous paragraph
  • The third one corresponds to a different topic

I think the same goal could be achieved with a less intrusive change. Something like this:

 You can download them from the [openSUSE Build
 Service](https://download.opensuse.org/repositories/systemsmanagement:/Agama:/Devel/images/iso/).
+Make sure to download the correct ISO file according to your system system architecture (eg.
+you would need to choose an file including `x86_64` if you use an Intel or AMD 64-bit processor)
+and according to the system you want to install (openSUSE vs ALP).

Writing a good README is much harder than it looks (there is a hard balance about what to include and what not). And maintaining it up to date is also very important (says a lot about the health of a project). So thank you sincerely for taking care and sorry for being a bit nitpicking. ;-)

@nagu165

This comment was marked as resolved.

README.md Outdated Show resolved Hide resolved
@dgdavid dgdavid merged commit 6b34668 into openSUSE:master Jan 19, 2024
@dgdavid dgdavid changed the title Update README.md [web] Update README with live ISO hints Jan 19, 2024
@dgdavid
Copy link
Contributor

dgdavid commented Jan 19, 2024

@nagu165

Thank you a lot for this contribution, already merged in master branch. Hope it helps people to choose the right ISO.

Thanks also to my @openSUSE/yast team colleagues for given their feedback here. Much appreciated.

@imobachgs imobachgs mentioned this pull request Feb 12, 2024
@imobachgs imobachgs mentioned this pull request May 17, 2024
imobachgs added a commit that referenced this pull request May 17, 2024
Prepare for releasing Agama 8. It includes the following pull requests:

* #884
* #886
* #914
* #918
* #956
* #957
* #958
* #959
* #960
* #961
* #962
* #963
* #964
* #965
* #966
* #969
* #970
* #976
* #977
* #978
* #979
* #980
* #981
* #983
* #984
* #985
* #986
* #988
* #991
* #992
* #995
* #996
* #997
* #999
* #1003
* #1004
* #1006
* #1007
* #1008
* #1009
* #1010
* #1011
* #1012
* #1014
* #1015
* #1016
* #1017
* #1020
* #1022
* #1023
* #1024
* #1025
* #1027
* #1028
* #1029
* #1030
* #1031
* #1032
* #1033
* #1034
* #1035
* #1036
* #1038
* #1039
* #1041
* #1042
* #1043
* #1045
* #1046
* #1047
* #1048
* #1052
* #1054
* #1056
* #1057
* #1060
* #1061
* #1062
* #1063
* #1064
* #1066
* #1067
* #1068
* #1069
* #1071
* #1072
* #1073
* #1074
* #1075
* #1079
* #1080
* #1081
* #1082
* #1085
* #1086
* #1087
* #1088
* #1089
* #1090
* #1091
* #1092
* #1093
* #1094
* #1095
* #1096
* #1097
* #1098
* #1099
* #1100
* #1102
* #1103
* #1104
* #1105
* #1106
* #1109
* #1110
* #1111
* #1112
* #1114
* #1116
* #1117
* #1118
* #1119
* #1120
* #1121
* #1122
* #1123
* #1125
* #1126
* #1127
* #1128
* #1129
* #1130
* #1131
* #1132
* #1133
* #1134
* #1135
* #1136
* #1138
* #1139
* #1140
* #1141
* #1142
* #1143
* #1144
* #1145
* #1146
* #1147
* #1148
* #1149
* #1151
* #1152
* #1153
* #1154
* #1155
* #1156
* #1157
* #1158
* #1160
* #1161
* #1162
* #1163
* #1164
* #1165
* #1166
* #1167
* #1168
* #1169
* #1170
* #1171
* #1172
* #1173
* #1174
* #1175
* #1177
* #1178
* #1180
* #1181
* #1182
* #1183
* #1184
* #1185
* #1187
* #1188
* #1189
* #1190
* #1191
* #1192
* #1193
* #1194
* #1195
* #1196
* #1198
* #1199
* #1200
* #1201
* #1203
* #1204
* #1205
* #1206
* #1207
* #1208
* #1209
* #1210
* #1211
* #1212
* #1213
* #1214
* #1215
* #1216
* #1217
* #1219
* #1220
* #1221
* #1222
* #1223
* #1224
* #1225
* #1226
* #1227
* #1229
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants