11
43
Fork
You've already forked base
6

Descript how people can contribute to this project #116

Merged
gerhardbeck merged 3 commits from duco/lerntools_base:duco-patch-1 into main 2022年05月21日 11:15:50 +02:00
Contributor
Copy link

Added a CONTRIBUTING.md that describes how people can contribute to this project. I added parts inspired by other projects. I tried to keep it short and simple, as there is currently not a lot of contribution.

Added a CONTRIBUTING.md that describes how people can contribute to this project. I added parts inspired by other projects. I tried to keep it short and simple, as there is currently not a lot of contribution.
Added a CONTRIBUTING.md that describes how people can contribute to this project. I added parts inspired by other projects. I tried to keep it short and simple, as there is currently not a lot of contribution.
The wiki contains helpful information for the development, so it is helpful to mention it here, so contributors can find it more easily.
Author
Contributor
Copy link

Dies ist ein erster Entwurf, der durch eine Diskusion auf Mastodon angeregt wurde. Ich bin offen für Verbesserungsvorschläge. Eine der großen offenen Fragen ist hier wohl auch die Thematik mit Deutsch und Englisch. Normalerweise werden projekte komplett in Englisch entwickelt. In diesem Projekt ist aber vieles auf Deutsch. Soll es da irgendwelche Regelungen geben?

Dies ist ein erster Entwurf, der durch eine Diskusion auf Mastodon angeregt wurde. Ich bin offen für Verbesserungsvorschläge. Eine der großen offenen Fragen ist hier wohl auch die Thematik mit Deutsch und Englisch. Normalerweise werden projekte komplett in Englisch entwickelt. In diesem Projekt ist aber vieles auf Deutsch. Soll es da irgendwelche Regelungen geben?
First-time contributor
Copy link

Danke für die Arbeit. Unsere Coder müssen noch draufschauen 😀

Von der Sprache her werden wir wohl erstmal zweisprachig bleiben: Wir sind bisher ein rein deutsches Team mit fast nur deutschen Besuchern.

Sinnvoll wäre: Code und Commits auf Englisch, Issues diskutieren wir auch gerne auf Deutsch

Danke für die Arbeit. Unsere Coder müssen noch draufschauen 😀 Von der Sprache her werden wir wohl erstmal zweisprachig bleiben: Wir sind bisher ein rein deutsches Team mit fast nur deutschen Besuchern. Sinnvoll wäre: Code und Commits auf Englisch, Issues diskutieren wir auch gerne auf Deutsch

Sinnvoll wäre: Code und Commits auf Englisch, Issues diskutieren wir auch gerne auf Deutsch

Dem schließe ich mich an.

> Sinnvoll wäre: Code und Commits auf Englisch, Issues diskutieren wir auch gerne auf Deutsch Dem schließe ich mich an.
Ryuno-Ki left a comment
Copy link

Wenn das drinne ist, sollten wir vielleicht auch eine deutsche Übersetzung bereit stellen (hier oder im Wiki)

Wenn das drinne ist, sollten wir vielleicht auch eine deutsche Übersetzung bereit stellen (hier oder im Wiki)
@ -0,0 +12,4 @@
* Stay respectful while contributing.
* Personal attacks are not allowed.
* Discrimination based on race/ethnic group, gender, age, religion, disability or sexuality is not allowed.
Member
Copy link

Ist das eine abschließende Liste?

Ist das eine abschließende Liste?
Author
Contributor
Copy link

Es ist eine Liste, von Dingen, die hier hoffentlich nicht toleriert werden. Ich weiß nicht, was "abschließend" in diesem Kontext bedeuten soll. Ich sehe das wie Gesetze in einem Staat. Da gibt es Gesetze und mit der Zeit können die sich ändern und neue hinzu kommen. Zu einem Zeitpunkt gilt ein bestimmtes Set an Gesetzen. Mit diesem PR schlage ich dieses Set an Regeln vor. Sollte es Probleme mit irgendwas geben, kann man natürlich neue einführen oder meinen Vorschlag überarbeiten.

Es ist eine Liste, von Dingen, die hier hoffentlich nicht toleriert werden. Ich weiß nicht, was "abschließend" in diesem Kontext bedeuten soll. Ich sehe das wie Gesetze in einem Staat. Da gibt es Gesetze und mit der Zeit können die sich ändern und neue hinzu kommen. Zu einem Zeitpunkt gilt ein bestimmtes Set an Gesetzen. Mit diesem PR schlage ich dieses Set an Regeln vor. Sollte es Probleme mit irgendwas geben, kann man natürlich neue einführen oder meinen Vorschlag überarbeiten.
First-time contributor
Copy link

Da stimme ich @duco zu: Es ist ein Anfang.

Da stimme ich @duco zu: Es ist ein Anfang.
CONTRIBUTING.md Outdated
@ -0,0 +16,4 @@
## Issues (Bugs and feature requests)
Please use the search function on the [issues page](https://codeberg.org/lerntools/base/issues) first, to check if the issue was already created by someone else. You can [write new Bug reports and feature requests](https://codeberg.org/lerntools/base/issues/new) in English or German. If you just have a question feel free to ask at [Mastodon](https://bildung.social/@lerntools).
Member
Copy link

*bug reports
(klein geschrieben)

*bug reports (klein geschrieben)
duco marked this conversation as resolved
@ -0,0 +39,4 @@
* Use the present tense ("Add feature" not "Added feature")
* Limit the first line to 72 characters or less
* Reference related issues and pull requests
* Describe the changes, so users and administrators will understand it (not "fix bug in App.vue", better "Stop navigation bar from flickers")
Member
Copy link

Wir sollten noch erwähnen, dass die Änderungen im Allgemeinen gegen dev erfolgen sollten.

Wir sollten noch erwähnen, dass die Änderungen im Allgemeinen gegen dev erfolgen sollten.
Author
Contributor
Copy link

Guter Punkt. Habe ich ergänzt.

Guter Punkt. Habe ich ergänzt.
duco marked this conversation as resolved
@ -0,0 +46,4 @@
* Half finished changes will not be merged. If you plan to further work on it wait until you are finished with the pull request or add "WIP:" in front of the title.
* The build command has to work without errors.
* Don't put code into comments. If code isn't needed anymore, delete it. If we like to look up the previous code we always can use the git history. There is no need keep it in comments.
* If you change the dependencies in the `package.json`, always run `yarn install` to add them to the `yarn.lock` file as well.
Member
Copy link

*yarn

(install sollte nicht notwendig sein, glaube ich)

*yarn (install sollte nicht notwendig sein, glaube ich)
Author
Contributor
Copy link

Das würde reichen, ja. Die Dokumentation von yarn sagt dazu:

Running yarn with no command will run yarn install, passing through any provided flags.

Ich würde jedoch empfehlen yarn install in der dokumentation zu schreiben, denn

  • es it explizit (dort steht also was passiert, man muss nicht wissen, dass dadurch etwas installiert wird)
  • man kann danach suchen (wenn man einfach nach yarn sucht ist es viel schwieriger heraus zu finden, was der Befehl macht. Wenn man nach yarn install sucht kommt man direkt zur Doku)
  • andere Projekte geben das auch so an, wie Mastodon
Das würde reichen, ja. Die Dokumentation von yarn sagt dazu: > Running `yarn` with no command will run `yarn install`, passing through any provided flags. Ich würde jedoch empfehlen `yarn install` in der dokumentation zu schreiben, denn * es it explizit (dort steht also was passiert, man muss nicht wissen, dass dadurch etwas installiert wird) * man kann danach suchen (wenn man einfach nach `yarn` sucht ist es viel schwieriger heraus zu finden, was der Befehl macht. Wenn man nach `yarn install` sucht kommt man direkt zur Doku) * andere Projekte geben das auch so an, wie [Mastodon](https://docs.joinmastodon.org/dev/setup/)
Ryuno-Ki marked this conversation as resolved
First-time contributor
Copy link

Sinnvoll wäre: Code und Commits auf Englisch, Issues diskutieren wir auch gerne auf Deutsch

Dem schließe ich mich an.

@duco kannst du die Sprachenfrage noch mit aufnehmen?

> > Sinnvoll wäre: Code und Commits auf Englisch, Issues diskutieren wir auch gerne auf Deutsch > > Dem schließe ich mich an. > @duco kannst du die Sprachenfrage noch mit aufnehmen?
Sign in to join this conversation.
No reviewers
Labels
Clear labels
Kind/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
Kind: Enhancement
improvement of a feature
Kind: Feature
Something totally new
Kind: Interesting for Wiki
to be saved in the wiki
Kind: invalid
Something is wrong
Kind: Question
More information is needed
Kind: Security
Relevant for security
Kind:bug
Something is not working
Priority: High
Priority: Low
Priority: Medium
Status: Contribution welcome
Contributions are very welcome, get started here
Status: Duplicate
This issue or pull request already exists
Status: External problem
There is an external problem
Status: Help wanted
Need some help
Status: In progress
Someone's working on it
Status: In review
we're testing a possible solution
Status: next release
All tested and ready for the next release
Status: Starters
This is a good task to start helping lerntools
Status: Tests neededd
Need some tests for locating the problem
Status: won't fix
This won't be fixed
vue3
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lerntools/base!116
Reference in a new issue
lerntools/base
No description provided.
Delete branch "duco/lerntools_base:duco-patch-1"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?