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

Leg uit dat pull requests op het normenkader niet worden geaccepteerd. #72

Closed
wants to merge 1 commit into from

Conversation

floort
Copy link
Contributor

@floort floort commented Mar 25, 2024

No description provided.

@floort
Copy link
Contributor Author

floort commented Mar 25, 2024

Ik weet niet wat "De plugin" is, maar als er iets niet werkt met een forked repo dan is dat handig om te documenteren voor alle externen. Bijdragen moet via pull requests en daarvoor moet eerst een fork gemaakt worden. Als er iets niet werkt in die forks kunnen bijdragen niet goed getest worden.

Edit: Dit was een reactie op een bericht van @berrydenhartog die nu is verwijderd.

@ruthkoole
Copy link
Collaborator

Hoi @floort ,

We hebben inmiddels gewerkt aan contributing guidelines voor het algoritmekader. Zie https://github.com/MinBZK/Algoritmekader/blob/main/CONTRIBUTING.md
Hierin hebben we onder andere ook uitgelegd dat het verwerken van een pull-request of issue soms even op zich kan laten wachten. Dat heeft ermee te maken dat het werken met Git nieuw is voor het team, en anderzijds dat we niet met alle bouwblokken tegelijkertijd bezig zijn. De bouwblokken worden één voor één opgepakt. Het is daarom soms logischer om een inhoudelijk punt pas op te pakken als er een goed team is om met dat onderwerp aan de slag te gaan.

Met het gebruik van de releases hopen we iets meer structuur te krijgen in het verwerken van de feedback. Al doende leert men, en hopelijk gaat het de komende tijd een stuk beter.

Hopelijk hebben we zo voldaan aan jouw suggestie voor verduidelijking op de werkwijze. Ik wil deze pull-request daarom sluiten.

@ruthkoole ruthkoole closed this Apr 11, 2024
@floort
Copy link
Contributor Author

floort commented Apr 11, 2024

De contributing guidelines lossen het onderliggende issue niet op. Ik lees in een comment onder #PR71 dat releases ongeveer eens per 6 weken zullen zijn. En deze laatste release was een opgespaarde bulk-commit. Met die aanpak kan je als je een bijdrage wilt leveren geen rekening houden met onzichtbaar parallel werk en dan loop je het risico dat PR's worden afgewezen op basis van een stukje structuur in plaats van inhoud.

Als alle commits gewoon zichtbaar zijn kunnen bijdragen worden geleverd op basis van de laatste stand van zaken en kan er meer respectvol met de tijd en bijdragen van externen worden omgegaan door ze niet op basis van een informatieachterstand werk te laten leveren. Het is overigens ook goed als die manier van bijdragen niet wordt ondersteund, maar dan is het wel respectvol om daar vooraf duidelijk over te zijn. Vandaar mijn voorstel om te documenteren dat een dergelijke aanpak niet wordt ondersteund.

Overigens kan ik me vinden in het sluiten van dit specifieke PR, als maar duidelijk is dat het nog niet is opgelost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants