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

Add CONTRIBUTING.md #276

Closed
wants to merge 1 commit into from
Closed

Add CONTRIBUTING.md #276

wants to merge 1 commit into from

Conversation

Potherca
Copy link
Member

@Potherca Potherca commented Dec 22, 2022

This is not finished yet, hence this MR is merged as "draft". Feedback on what has already been written is welcome.

The "Release" section is a continuation of the list from #248

Not to forget:

@Potherca Potherca self-assigned this Dec 22, 2022
@Potherca Potherca added this to the v2.6.0 milestone Dec 22, 2022
@Potherca Potherca mentioned this pull request Dec 22, 2022
7 tasks
@kirchsth
Copy link
Member

Hi @Potherca,

I like the idea.

What do you think if we directly define a "Release (issue) template" and defined there all checks which should be done. And the constributing.md contains only a reference to the template.
A generic issue template could be added and referenced too.

Best regards
Helmut

@kirchsth
Copy link
Member

I found this CONTRIBUTING.md and the corresponding CODE_OF_CONDUCT.md and other links, ...

It contains a lot of guidelines, e.g "Write good commit messages". Or it contains a description of the folder structure, ....

@Potherca
Copy link
Member Author

Potherca commented Dec 25, 2022

In my experience it helpt to have both the prose in the CONTRIBUTING.md and the template, but yes, I was also thinking about issue templates!

I don't like the defaults GitHub offers, so I was thinking about defining what it is we want first, and then creating templates from that.

This contributing file is from another FOSS project I am part of. I tend to add a directory structure as wel (see here for an example) as it helps get a quick overview. So, yeah, the contributing you linked to is similar to the end goal here. 👍

For the code of conduct I would rather reach a consensus at the plantuml-stdlib organisation level. We'd probably have to open a discussion at the org level for that. I've just not had the bandwidth for that yet. (The current focus is on getting a few release out the door first, getting the PlantUML lib updated, and then focus on the rest of the org, automation, etc.).

As for commit messages, I'm not that bothered, honestly. At this point I think the contributions themselves are more valuable than how they are described in the commit message. However there is no reason why you and I shouldn't start by defining at least some order to our madness. I already have some conventions I follow and I've been party to several more rigid regimes for various companies I've worked with. That might also be worth starting a discussion at the org-level, I think...

@kirchsth
Copy link
Member

I see you have much more experiences in this area than I have. You are right we should start with smaller steps.

Related to the commit messages, I only saw it and thought it is a very complete description which covers nearly everything. Personally I'm happy if I can check in some files into git (without deleting other local files during a merge process ;-) )

@Potherca
Copy link
Member Author

Personally I'm happy if I can check in some files into git (without deleting other local files during a merge process)

Yeah, I keep coming back to this:

Git may be simple but it is not easy.

The highest voted questions on StackOverflow:

  • Top 3: 60% git
  • Top 10: 50% git
  • Top 25: 40% git
    image

With the other almost 40%1 of the Top 25 (marked blue) is taken up by javascript and JSON.

Footnotes

  1. 36% to be precise

@stale
Copy link

stale bot commented Mar 11, 2023

This issue has been automatically marked as stale because it has not had activity in the past 60 days. It will be closed in seven days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Issue has not been active for 60 days label Mar 11, 2023
@Potherca Potherca added the not-stale Stop issue from being marked stale by bot label Mar 11, 2023
@stale stale bot removed the stale Issue has not been active for 60 days label Mar 11, 2023
@kirchsth kirchsth removed this from the v2.6.0 milestone Apr 22, 2023
@kirchsth kirchsth removed the not-stale Stop issue from being marked stale by bot label Mar 17, 2024
@kirchsth
Copy link
Member

cleanup stalled pull request.
If still a concrete solution is planed please reopen it or create a new issue/pull-request.

@kirchsth kirchsth closed this Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants