TeleStrat...
Closed this issue · 6 comments
Hello Laurent,
J'ai ajouté l'exécution des disques StratSed sous TeleStrat. Il ne s'agissait que de quelques lignes à ajouter et je pratique le C# depuis quelque temps.
Par contre je débute sur Github, je ne sais pas comment envoyer ce que j'ai fait.
@+
damien (un ancien Oricien).
Bonjour Damien, merci beaucoup de ton message et désolé de cette réponse super tardive, je consulte rarement l'email associé à ce compte GitHub en fait !!!
C'est super sympa de ta part d'avoir ajouté une nouvelle fonctionnalité à cet outil, merci beaucoup !
Pour réintégrer ta contribution sur mon dépôt d'OricExplorer, le mieux sertait que tu fasses une "pull request", que je pourrai ensuite accepter.
Pour être franc avec toi, je ne suis pas non plus un pro de Git ni de Github, je n'ai jamais fait moi-même de pull requests sur Github, mais j'ai trouvé un tutorial qui semble assez simple:
https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request-from-a-fork
On peut essayer ça, et on verra ce que ça donne :-)
A+
Laurent
ps - je vais surveiller mon mail et ce compte Github plus régulièrement maintenant, pour éviter de te répondre 1 mois après !!
Salut Laurent,
Merci pour ta réponse. Il faut donc que je forke le projet ? Actuellement il est juste cloné en local sur ma machine...
@+
damien.
Re,
Voilà, j'ai forké/pushé et fait la pull request...
@+
damien.
Bonsoir Damien, et désolé encore pour la réponse tardive!!!
En fait, tu n'avais pas forcément besoin de forker le projet pour faire une pull request, je n'avais pas vérifié si tu avais fait un fork ou pas... En fait la manière de faire une pull request est de mon point de vue un peu moins facile si tu ne forkes pas, et le "tuto" n'est pas exactement le même...
Bref, j'ai regardé tes modifs, que je vais accepter, je sui sjuste un peu surpris qu'il n'y ait pas plus de modifs de code pour gérer les disques Stratsed, j'aurais pensé que le format des disques aurait pu être différent des disques Sedoric de l'Atmos et aurait pu entraîner des modifs plus conséquentes dans le code...
J'ai vu que tu avais également légèrement retouché/nettoyé le fichier projet .csproj, notamment pour supprimer la dépendance au fwk .Net v2.0...
Je n'ai pas compris en revanche pourquoi tu avais changé "GenerateManifests" de true à false, mais je ne connais pas assez .Net pour savoir de quoi il retourne exactement...
Bref, je valide la pull request, merci encore pour ta contribution !!!
Es-tu d'accord pour que je mentionne ton nom comme contributeur dans la doc (le fichier Readme.md ?)
Salut Laurent,
Je n'ai aucun mérite : Scott avait fait tout le boulot de gestion du format des disques. Le seul truc qui restait à faire c'était de lancer l'émulateur avec le "bon" paramètre. Ce qui n'était pas le cas puisque c'était l'émulateur Atmos qui était invoqué inconditionnellement.
Concernant les modifs du .csproj elles avaient initialement pour but de désactiver le déploiement (ClickOnce) qui n'était apparemment pas exploité. La suppression de la dépendance du FW .NET 2.0 n'est pas une opération volontaire, elle est probablement imputable à la conversion du projet en VS 2019. Même si mes modifs ne justifient pas cette montée de version du FW pré-requis, je ne pense pas que cela pose problème. Au pire on pourra revenir en arrière facilement.
La désactivation du GénérateManifest et liée à la désactivation de ClickOnce. Pour cette fonctionnalité le fichier Manifest doit être créé hors de l'exécutable. Lorsque ClickOnce est désactivé, ce fichier peut-être "internalisé" dans l'exécutable. Je pense que moins il y a de fichiers en dehors de l'exécutable, moins il y a de risques de dysfonctionnement.
Pour répondre à ta dernière question, fais comme tu veux. Vu le peu de modifs que j'ai faites, personnellement j'ai trouvé que ce n'était pas justifié d'ajouter mon nom dans la fenêtre About. Mais si tu veux m'ajouter dans le readme.md ça ne me pose aucun problème.
D'ailleurs ça me fait penser que je n'ai même pas changé le n° de version... du coup ça va peut-être manquer de visibilité pour les éventuels utilisateurs.
Pour finir, dans mon dépôt j'ai créé une nouvelle release pour mettre à dispo le nouveau .zip mais je ne suis pas sûr d'avoir fait ça correctement...
@+
damien.
Je crois qu'on peut fermer cette issue qui est traitée...