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

Code of conduct fehlt #32

Closed
Huluvu424242 opened this issue Nov 12, 2019 · 3 comments · Fixed by #34
Closed

Code of conduct fehlt #32

Huluvu424242 opened this issue Nov 12, 2019 · 3 comments · Fixed by #34
Assignees
Labels
bug Something isn't working
Milestone

Comments

@Huluvu424242
Copy link
Owner

Huluvu424242 commented Nov 12, 2019

Describe the bug
Der Link in der Readme auf die Code of Conduct Guidelines ist verwaist.

To Reproduce
Klick auf den Link :)

Expected behavior
Die an das Projekt angepassten Guidlines erscheinen - angepasste eMail etc.

[Update 19.11.2019]
Ticket: issue32
Umsetzung: PR#34

@Huluvu424242 Huluvu424242 added the bug Something isn't working label Nov 12, 2019
@Huluvu424242 Huluvu424242 self-assigned this Nov 12, 2019
@Huluvu424242 Huluvu424242 added this to Backlog in Jenkins Monitor Board via automation Nov 12, 2019
@Huluvu424242 Huluvu424242 added this to the 0.0.2 milestone Nov 12, 2019
Huluvu424242 added a commit that referenced this issue Nov 13, 2019
@jdippel
Copy link
Collaborator

jdippel commented Nov 18, 2019

Die Beschreibung ist unklar.
Im Absatz Unterstützung zur Beschreibung von jenkinsmonitor wird auf zwei Verhaltensregeln via Link verwiesen, wobei für einen der beiden Links das Ziel nicht definiert ist.
Der eine (funktionierende) Link ruft eine Seite von außerhalb auf, der nicht funktionierende könnte ein lokales Abbild dieser Seite aufrufen.
Demnach müsste der Inhalt der Seite nur lokal gespeichert werden.

@jdippel
Copy link
Collaborator

jdippel commented Nov 18, 2019

In Ticket #24 wird darauf verwiesen, dass der Text unter src/site/resources/code-of-conduct.txt liegen soll. Die angegebene Referenz ist aber nicht vorhanden.

@Huluvu424242
Copy link
Owner Author

Du meintest vermutlich issue32 (dieses hier), mich verwirren die vielen Zahlen auch immer. Daher hab ich gerade mal die Issue Beschreibung aktualisiert.
issue = 32
PR = 34
Im Pull Request 34 wird das Issue 32 behoben. Zumindest ist das das Ziel. Die fehlende Datei muss meiner Meinung nach auch (wie Du beschrieben hast) unter src/site/resources/code-of-conduct.txt eingecheckt werden. Durch die Maven Site Generierung sollte sie dann unter docs landen und nach Merge des PR in den Master sollte sie dann per github Verlinkung von der README aus erreichbar sein. Das lässt sich leider nicht wirklich vorher testen - nur lokal mal bauen und ausprobieren mehr geht halt nicht.
Das docs Verzeichnis im master entspricht praktisch der Webseite des Projektes. Diese wäre sonst auch über diesen Link erreichbar: https://funthomas424242.github.io/jenkinsmonitor/

Generell ist die extern gehostete Datei ein Template bei dem lediglich die Platzhalter gegen Werte zu ersetzen sind, wenn man sie ins Projekt übernimmt. Das habe ich jetzt getan (im PR 34).
Bitte mal drauf schauen ob ich noch einen Platzhalter vergessen habe oder das ich sie wirklich an die richtige Stelle gelegt hab. Ich hab zwar schon drüber geschaut aber 4 Augenprinzip eben - wäre schön :)

Danke

@Huluvu424242 Huluvu424242 moved this from Backlog to In progress in Jenkins Monitor Board Mar 4, 2020
@Huluvu424242 Huluvu424242 linked a pull request Mar 4, 2020 that will close this issue
Jenkins Monitor Board automation moved this from In progress to Done Mar 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

2 participants