search

varnishcache / homepage

The project homepage
https://varnish-cache.org/
21 stars 58 forks source link

Add organization of Varnish Cache Project #80

Closed hermunn closed 4 years ago

hermunn commented 5 years ago

Coding style guidelines has been deliberately not moved from the Varnish Cache doc tree, because I want this to be focused on organization and governance.

hermunn commented 5 years ago

I forgot to force push before making the pull request. Please go to Files changed and comment there if there is anything. Nils' comments on the first version have all been amended.

hermunn commented 5 years ago

I have tried to write something sensible about feature requests, and I think what I wrote is close to reality, or what we want reality to be.

nigoroll commented 5 years ago

@Dridi as much as I agree with your comment, I think it would help if we stayed focused on the overall organizational issues. I would expect your suggestions for changes to be uncontroversial.

bsdphk commented 5 years ago

@Dridi while I agree in principle, I find "otherwise we will not hesitate to close your issue" a little harsh.

hermunn commented 5 years ago

I have pushed another commit that will be squashed before merge, and it includes some dead links to a non-existing Contact Us page, which, in my humble opinion, should be created before this is merged.

There is also a possible contradiction about the commit bit, and how still nothing about how disputes are recorded and resolved. I am not sure I am up to the task of suggesting something here on my own, so maybe we should merge this and discuss this part in some other forum.

hermunn commented 5 years ago

A new fixup commit has been pushed, and I think things look good. However, the Contact Us page is not present, and I think that should be merged first.

hermunn commented 5 years ago

I will probably not be able to update the PR this week, so if someone wants to take over, feel free. I think that contributors can actually push to this branch, but I have been wrong before. I rivaling PR which squashes my commits and some extra proposed changes is also an option. That will make it easier to comment, too. In that case, remember to rebase and link to the correct Contact Us page.

nigoroll commented 5 years ago

@hermunn never mind the squash comment, I was lagging behind with assumptions

hermunn commented 4 years ago

Rebased, fixed a typo, and added a commit with new formulation as suggested on IRC. The two commits should be squashed before the merge.

hermunn commented 4 years ago

Everybody, please let's get this ratified. If anything pops up, we can still make more changes later.

I agree. I have acted upon all the changes I have found in comments, and think it is time. If people needs an extra day for a final review, that is fine, but let's not forget about this now.