Ash Berlin-Taylor comments

Results 204 comments of


Ash Berlin-Taylor

Upgrading to airflow 2.4.0 from 2.3.4 causes NotNullViolation error

Argh! Good catch

Switch to using `pip.astronomer.io` as primary index

Not needed, we have reserved the package names on Pypi

Add runner-level config to enforce PR security

@chrispat Tagging you since you commented on #494. I agree that solving this at the GitHub level would be a better solution, but on apache/airflow we are struggling without being...

Add runner-level config to enforce PR security

(Oh and of course this will need tests adding.)

Add runner-level config to enforce PR security

Question: should these security settings also apply to branch builds?

Add runner-level config to enforce PR security

@pietroalbini Would love to collaborate on this. Does this PR do everything you need for Rust?

Add runner-level config to enforce PR security

Gotcha, yes that makes sense. I'll roll that in to my PR.

Add runner-level config to enforce PR security

Having now started using this "in anger" the current config scheme I have in this PR is clearly not right.

Add runner-level config to enforce PR security

I've just run in to a less-than ideal behaviour (which is likely only a problem for the `apache` org and any other umbrella orgs): the `MEMBER` author_association wins out over...

Add runner-level config to enforce PR security

> Nevertheless, do you have any resource where you publish own/separate builds of actions/runner with your PR modification? This would allow to use these instead of the GitHub version of...