Skip to content

Conversation

@eladkal
Copy link
Contributor

@eladkal eladkal commented Jul 12, 2025

No description provided.

@eladkal eladkal requested a review from potiuk July 12, 2025 07:15
@potiuk
Copy link
Member

potiuk commented Jul 12, 2025

Hmm. I wonder why dependabot did not do it automatically (it should) - maybe we should reconfigure it to bump things "always" when they are released and not only when there are security fixes (for build-requirements) - not sure if we can do it selectively ?

@eladkal
Copy link
Contributor Author

eladkal commented Jul 12, 2025

I wonder why dependabot did not do it automatically (it should)

Not only here. you mentioned also relying on dependabot for datamodel-code-generator #47172
that also doesn't work

@eladkal eladkal merged commit ace42ad into apache:main Jul 12, 2025
170 checks passed
@eladkal eladkal deleted the pluggy branch July 12, 2025 08:49
@potiuk
Copy link
Member

potiuk commented Jul 12, 2025

I wonder why dependabot did not do it automatically (it should)

Not only here. you mentioned also relying on dependabot for datamodel-code-generator #47172 that also doesn't work

@gopidesupavan -> wdyt ? I think the change you contributed to dependabot shouls bump such versions automatically (I looked at our configuration and we use default "applies-to" version not "security-check" so it should work I think,

@potiuk
Copy link
Member

potiuk commented Jul 12, 2025

Ah... so we actually had the dependabot PR #50660 but we did not merge it !

@potiuk
Copy link
Member

potiuk commented Jul 12, 2025

So I think we need to be a little more careful about not missing those PRs :)

@gopidesupavan
Copy link
Member

Yeah, we should keep an eye :)

And datamodel-code-generator its under dependency-groups, i thoguth dependaboat supports that already, otherwise i could have pushed fix to dependaboat :(, good we have already a PR raised i think that should solve. #47172 (comment)

stephen-bracken pushed a commit to stephen-bracken/airflow that referenced this pull request Jul 15, 2025
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

Successfully merging this pull request may close these issues.

3 participants