django-typer

[williln]

Hi @django-commons/django-typer! I wanted to take a moment and check in with you about how the transition to Django Commons has been.

• Have you run into any friction having your project in Django Commons, if so, can you describe them?
• Is there anything we can do to help your team or repo out?
• Do you have any questions or concerns for us?

If you'd like to chat with us privately, our email is [email protected]. Thanks!

[bckohan]

• Have you run into any friction having your project in Django Commons, if so, can you describe them?

It's been smooth! I tweaked some of the base GA scripts a little. I have one that checks to see if the CoC has updated once a week and if it has it opens a PR to pull in the new CoC. I did this because if it changes I wanted some automatic notification as to what the changes were. I also tweaked the release GA a bit but gave up trying to get it to do what I want it to do which is run my tests first and only publish if everything passes. There seems to be some conditional flow control limitations wrt to how a GA can run other workflows.

• Is there anything we can do to help your team or repo out?

Not anything urgent! It'd be nice if the CoC check could be a push (webhook) instead of a pull but ¯\(ツ)/¯

• Do you have any questions or concerns for us?

I'm self conscious about how long the CI runs on this repo (~20 minutes). There's no way to avoid starting up a bunch of python subprocesses to test this library because of what it is. Every time this happens all the pytest/coverage stuff has to bootstrap which takes awhile so there's not much more efficiency I can wring out of it. I'm worried it might be an unnecessarily large hog on the actions quota. I'm not sure how concerned I should be about this. I'm considering disabling CI on commits and only running it on PRs.

[tim-schilling]

if the CoC check could be a push (webhook) instead of a pull

That's an interesting idea! I do hope that if we make a change to the CoC there'd be a discussion post here tagging all members. But we haven't run into that use case, so I'm not sure we can promise that.

I'm worried it might be an unnecessarily large hog on the actions quota

For now, this isn't a problem. Public repos don't have quotas as far as I know. But it's nice to reduce run-times for the environment 💚

[cunla]

if the CoC check could be a push (webhook) instead of a pull

I noticed that GitHub workflow, it is very cool! However, I think it is a bit of overkill, a simple link to the django-commons CoC would suffice.

[bckohan]

I thought about just linking, but I'm a pedant and not having the full CoC available when clicking the CoC tab bothered me. And also, I wanted an automagic notification with a nice diff to see when and what any updates were. Not because I don't trust the updates, mostly because I figure if there is an update it was precipitated by something and I'd like the push to figure out what that was.