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

chore: add funding.json #5581

Merged
merged 2 commits into from
Aug 30, 2024
Merged

chore: add funding.json #5581

merged 2 commits into from
Aug 30, 2024

Conversation

AgeManning
Copy link
Contributor

Description

This adds a FUNDING.json file to the main repo. This is required for various projects, however the immediate case is for registering for an optimism retropgf grant.

@AgeManning AgeManning requested a review from jxs August 30, 2024 02:42
@guillaumemichel guillaumemichel changed the title Add funding.json chore: add funding.json Aug 30, 2024
@guillaumemichel
Copy link
Contributor

guillaumemichel commented Aug 30, 2024

There is a conflicting funding.json introduced in #5553 by @dhuseby, and IIUC potential grants would go the the Libp2p Core Fund.

I suggest to override funding.json. Is there a requirement on the filename (funding.json vs FUNDING.json)?

cc: @jxs @dariusc93 @dhuseby @p-shahi @yiannisbot @MarcoPolo @aschmahmann

@AgeManning
Copy link
Contributor Author

Oh Sorry. I didn't realise there was already another PR for this.

My understanding is that other libp2p maintainer teams are applying for the retro grant funding independently.

As we (Sigma Prime) have not received any funding for our new role as maintainers of rust-libp2p and we do plan to maintain this indefinitely, we had planned on applying for the retro funding which primarily covers the period since nucleation to help sustain this project long term.

Any funding obtained, we do plan to distribute for past work and solely to rust-libp2p and related projects.

@guillaumemichel
Copy link
Contributor

@AgeManning I don't have write permission on your branch. Could you please remove the older funding.json?

@jxs jxs added the send-it label Aug 30, 2024
@mergify mergify bot merged commit 95f40ff into libp2p:master Aug 30, 2024
72 checks passed
@dhuseby
Copy link
Contributor

dhuseby commented Aug 30, 2024

@AgeManning so what happened here? Did my funding.json get removed? I understand that independent maintainers are all applying separately. We need to figure out how to do this so that we can all do that.

@dhuseby
Copy link
Contributor

dhuseby commented Aug 30, 2024

Having merge wars over whose funding.json file gets into the repo is ridiculous.

@AgeManning
Copy link
Contributor Author

@dhuseby - Apologies, didn't intend on having any merge wars.

I thought we didn't have any funding.json when I made this PR (I was looking for the capital version which is used in other repos).

I notice now you already made a PR to add this, with an OP projectId. Have you applied for an OP retro grant for rust-libp2p?

Happy to line up a discussion about this if that is the case

@dhuseby
Copy link
Contributor

dhuseby commented Sep 6, 2024

@AgeManning I think this has all been sorted.

TimTinkers pushed a commit to unattended-backpack/rust-libp2p that referenced this pull request Sep 14, 2024
## Description

This adds a FUNDING.json file to the main repo. This is required for
various projects, however the immediate case is for registering for an
optimism retropgf grant.

---------

Co-authored-by: João Oliveira <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants