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

Implement Jazzband guidelines for django-invitations #172

Closed
12 tasks done
jazzband-bot opened this issue Mar 30, 2022 · 27 comments
Closed
12 tasks done

Implement Jazzband guidelines for django-invitations #172

jazzband-bot opened this issue Mar 30, 2022 · 27 comments
Assignees

Comments

@jazzband-bot
Copy link

jazzband-bot commented Mar 30, 2022

This issue tracks the implementation of the Jazzband guidelines for the project django-invitations

It was initiated by @bee-keeper who was automatically assigned in addition to the Jazzband roadies.

See the TODO list below for the generally required tasks, but feel free to update it in case the project requires it.

Feel free to ping a Jazzband roadie if you have any question.

TODOs

Project details

Description Generic invitations app for Django
Homepage
Stargazers 362
Open issues 44
Forks 140
Default branch master
Is a fork False
Has Wiki True
Has Pages False
@MrCordeiro
Copy link
Contributor

@valberg, please stop accepting PRs that don't pass the testing workflow

image

@valberg
Copy link
Contributor

valberg commented Mar 30, 2022

@MrCordeiro They do pass the testing flow if we merge in #174 :)

@valberg
Copy link
Contributor

valberg commented Apr 1, 2022

I have just created a Read the Docs project: https://readthedocs.org/projects/django-invitations/ and added jazzband-bot as maintainer.

@valberg
Copy link
Contributor

valberg commented Apr 4, 2022

Hey @bee-keeper

When you get the time it would be great if you could "Add jazzband account to PyPI project as maintainer role" as per the checklist above :)

@valberg
Copy link
Contributor

valberg commented May 31, 2022

@bee-keeper ping about the above :)

@LukasKlement
Copy link

Do you have an ETA for the next release? Django 4 is out for a while now, and this package is still not compatible.

@valberg
Copy link
Contributor

valberg commented Jun 27, 2022

@LukasKlement no, there is no ETA. When we get the pypi maintainer role assigned to the jazzband account things should start rolling.

For now I can recommend that you use the latest git sha

@valberg
Copy link
Contributor

valberg commented Jul 5, 2022

@bee-keeper Sorry to ping you again - but I think the last thing we need is to "Add jazzband account to PyPI project as maintainer role". Let me know if I can help with anything.

@valberg
Copy link
Contributor

valberg commented Aug 8, 2022

@bee-keeper Ping :)

@bee-keeper
Copy link
Member

Hi, have you some instruction to do this? is it just an email?

@MrCordeiro
Copy link
Contributor

@bee-keeper, you have to go to the project's page on PyPI (https://pypi.org/manage/project/django-invitations/collaboration), where you are currently the sole maintainer, and add the jazzband user (https://pypi.org/user/jazzband/) to the project. That way, the bot can send any new package release to PyPI.

Only the current owners of a project have the ability to add new owners or maintainers.

@bee-keeper
Copy link
Member

bee-keeper commented Aug 8, 2022

@MrCordeiro ok done, thanks

@MrCordeiro
Copy link
Contributor

Awesome!

@valberg, there's a CI workflow for project releases, but I think that if you create a new release with the latest changes now it will return a 500 error until the release key is added to the Repo's secrets (at least this was what happened here).

After that's done, the release workflow should send the project lead(s) and roadie an email to release the new version to PyPi.

And then' it's done!

@valberg
Copy link
Contributor

valberg commented Aug 8, 2022

Great! I'm guessing that adding the release key is not something I can do? If so, got any pointers @MrCordeiro ?

Btw. I do not have access to "settings" (https://github.com/jazzband/django-invitations/settings) - but I guess that is by design @jezdez ?

@Natureshadow
Copy link

Can we please either get this done, or make a release as it was done before? As it stands, Jazzband is currently preventing proper maintenance instead of helping it.

@valberg
Copy link
Contributor

valberg commented Sep 12, 2022

@jezdez @MrCordeiro I'm kinda blocked here - could you guide me in the right direction?

@MrCordeiro
Copy link
Contributor

MrCordeiro commented Sep 12, 2022

@valberg, if the release workflow is not working, I believe you need @jezdez to set the JAZZBAND_RELEASE_KEY (yes, we don't have access to the settings, but it's by design).

Not sure whether you can tick the last box yet or if you need to open a ticket first. I can interpret the bot's TODOs both ways.

Edit: just editing this to say that I appreciate the time you're investing in this @valberg (o゜▽゜)o ☆ !

@valberg
Copy link
Contributor

valberg commented Sep 28, 2022

I just tried to release 2.0.0 - but it failed: https://github.com/jazzband/django-invitations/actions/runs/3141681027/jobs/5104396248

Any ideas @MrCordeiro @jezdez ?

@jezdez
Copy link
Member

jezdez commented Sep 28, 2022

Hey all, sorry for the slow response, I've just added the JAZZBAND_RELEASE_KEY to the repo and this should be good to go.

@valberg and @brylie Please make sure to read https://jazzband.co/about/releases, https://jazzband.co/about/guidelines and https://jazzband.co/about/faq. I've restarted the 2.0.0 release action.

@jezdez
Copy link
Member

jezdez commented Sep 28, 2022

Oh bummer, turns out https://github.com/jazzband/django-invitations/pull/179/files#r982067309 basically broke the release workflow and doesn't actually publish to the Jazzband index and just configures it. This is independent of the lack of secret key above though. Please request a code review if you change the release workflow the next time.

@jezdez
Copy link
Member

jezdez commented Sep 28, 2022

I've opened #195 to fix it.

@jezdez
Copy link
Member

jezdez commented Sep 28, 2022

As the next steps:

  • get Fix release workflow to actually publish #195 merged
  • delete 2.0.0 tag
  • create a new 2.0.0 tag, which triggers the release workflow and uploads it to the Jazzband package index
  • @valberg and @brylie then get emails to their verified email addresses, requesting to verify and confirm the release via the website, just follow the instructions in the email. This is done to prevent malicious releases as anyone can create git tag in Jazzband
  • close this ticket (I think?)
  • celebrate 🥳

@valberg
Copy link
Contributor

valberg commented Sep 28, 2022

@brylie I guess we have to update the version in pyproject.toml to get the correct release build on jazzband.co - right now it is building 2.0.0. Do you wish to do it?

@brylie
Copy link
Member

brylie commented Sep 28, 2022

I will be AFK for a while, but I am available to approve a PR via my phone. Would you bump the release version?

@valberg
Copy link
Contributor

valberg commented Sep 28, 2022

@brylie I opted for removing the tags/releases and stick to 2.0.0 :)

@valberg
Copy link
Contributor

valberg commented Sep 28, 2022

There we go! 2.0.0 is out! https://pypi.org/project/django-invitations/

@valberg valberg closed this as completed Sep 28, 2022
@valberg
Copy link
Contributor

valberg commented Sep 28, 2022

Thanks of all the work @bee-keeper @jezdez @MrCordeiro @brylie

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

No branches or pull requests

8 participants