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

Create maintainers-minutes-2025-01-31.mdx #3025

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Original file line number Diff line number Diff line change
@@ -0,0 +1,95 @@
---
title: 'Maintainers Minutes: January 2025'
subtitle: 'Keeping you informed of the latest maintainers discussions'
pubDate: 2025-01-31T12:00:00+01:00
headerImage: '/assets/images/blog/maintainers-minutes-2024-07-26/maintainers_duck.png'
headerImageAlt: 'Cartoon yellow rubber duck with nf-core logo badge on its body with the nf-core logo.'
headerImageDim: [250, 250]
embedHeaderImage: true
authors:
- 'maxulysse'
label:
- 'maintainers'
---

import Profile from '@components/GitHubProfilePictureExtended.svelte';
import { Image } from 'astro:assets';

The 'Maintainers Minutes' aims to give further insight into the workings of the [nf-core maintainers team](/governance#maintainers)
by providing brief summaries of the monthly team meetings.

## Overview

In the January meeting we discussed (amongst others) the following topics:

- [Spring cleaning](#spring-cleaning)
- [References](#references)
- [stubs](#stubs)
- [CI with latest-everything](#ci-with-latest-everything)
- [Helpdesk Americas](#helpdesk-americas)
- [Default prefixes in modules](#default-prefixes-in-modules)
- [Best practices for spring cleaning](#best-practices-for-spring-cleaning)

## Spring cleaning

We decided that the spring cleaning that we planned to do before the hackathon should be done two weeks before the hackathon.
So it will happen from March 10th to March 14th.
We decided to extend the duration of the spring cleaning to a full week to give everyone enough time to participate.
The idea is to clean up issues, PRs, have a look at the repos and open new issues if needed.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a expand with a bit more datail, the community might be interested.

E.g. stuff like: we go through common repos, and close PR that are older than X years (After giving a warning), deleting merged branches, we check slack/github teams to make sure they are up to date, archiving pipelines


## References

Maxime Garcia wanted to know about the usage of `conf/igenomes.config`, and how it was actually used in the pipelines.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

People do use local cloens of igenomes at least... but I thin ktaht config is the remote onee right?

Since no one is really actually using it, he will move forward with his ideas, and will continue to work on his current POC.

> Whenever Maxime says “I can do whatever I want” I get really scared.

## Stubs

The discussion about stubs was going a bit in circles and off topic.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What was the discussion about? Have a leading sentence or a bette rtitle

It all started because the Nextflow language server was complaining about unused args in the stub section of any modules.
So people started to remove them, but then we have a stub block that is widely different from the script block, and no really way of knowing if the args are passed correctly within stubs.

We agreed that for now:

- We will echo args for now instead of removing them
- Talk with Ben, in particular in the static type PR and see if we can be smarter about it

## CI with latest-everything

We widely agreed that since `latest-everything` can break a pipeline, it is important to test with it, because we want our code to be up to date with the latest update.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is it breaking? Give more context

But we also agreed that we don't want failure in this tests to block PRs.
The end idea is that we keep the `latest-everything` tests in the CI workflow, but we don't require them to merge the PR.

## Helpdesk Americas

We discussed the helpdesk in the Americas timezone.
Since the start of the year, Florian and Lorena have mainly been on their own with very little audience.
We thought about ideas to improve it, and came up with the following:

- Maybe change time, and delay it a bit
maxulysse marked this conversation as resolved.
Show resolved Hide resolved
- Pump up on the communication, and make it more visible
- Explaining clearly what the helpdesk is, and that there are clearly no stupid questions
- See with the outreach team if we can get more visibility

Since we have all our sessions planed ahead, we can maybe communicate about it in a more efficient manner, as we currently rely on the people hosting the helpdesk to communicate about it.

## Default prefixes in modules

Our amazing modules aficionado Simon kindly reminded us about [this PR](https://github.com/nf-core/website/pull/2608).
It was decided that four maintainers should review it, so that we can hopefully close the discussion and advance on this topic and merge the PR.

## Best practices for spring cleaning

We discussed Spring Cleaning once more, and our relentless modules dark knight Simon shared a nice idea:

> We should check our open PRs and close them if they are no longer relevant.
maxulysse marked this conversation as resolved.
Show resolved Hide resolved

It would be nice indeed to keep track of all our open PRs, and issues and close them if we can.
That would simplify the spring cleaning, and all of our Justice League of maintainers' own lives.

## The end

As always, if you want to get involved and give your input, join the discussion on relevant PRs and slack threads!

\- :heart: from your #maintainers team!
Loading