Skip to content

Add dependaboat to check upgrades daily - #44966

Merged
jscheffl merged 5 commits into
apache:mainfrom
gopidesupavan:add-dependaboat
Dec 17, 2024
Merged

Add dependaboat to check upgrades daily#44966
jscheffl merged 5 commits into
apache:mainfrom
gopidesupavan:add-dependaboat

Conversation

@gopidesupavan

Copy link
Copy Markdown
Member

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@gopidesupavan

gopidesupavan commented Dec 16, 2024

Copy link
Copy Markdown
MemberAuthor

We can configure if any prefix required in PR title. ?

currently it doesn't support the cron expression, so have given daily.

We have dependency in pre-commit , but unfortunately dependaboat is not supported to bump dependencies in pre-commit.

Comment thread.github/dependabot.yml Outdated
Comment thread.github/dependabot.yml Outdated
gopidesupavanand others added 2 commits December 16, 2024 19:24
Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
Comment thread.github/dependabot.yml Outdated
Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>

@jscheffljscheffl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@amoghrajeshamoghrajesh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1
I say lets merge it and we can test it in real time if it works. Good job!

Comment thread.github/dependabot.yml
Comment thread.github/dependabot.yml

@potiukpotiuk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - but added one more comment about adding ./dev/breeze as well.

@jscheffl
jscheffl merged commit 50c2c9a into apache:mainDec 17, 2024
@gopidesupavan

Copy link
Copy Markdown
MemberAuthor

Thanks for the merge :) was bit of on travel today. it looks like dependaboat already started playing his role. hahah

@gopidesupavan

Copy link
Copy Markdown
MemberAuthor

@gopidesupavan

Copy link
Copy Markdown
MemberAuthor

LGTM - but added one more comment about adding ./dev/breeze as well.

Thanks for pointing missed that :)

@gopidesupavan
gopidesupavan deleted the add-dependaboat branch December 21, 2024 05:24
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
* add dependaboat to check upgrades daily
* add task_sdk to dependaboat directories list
* Update .github/dependabot.yml
Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
* Remove quotes
Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
* Update .github/dependabot.yml
---------
Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@gopidesupavan@potiuk@amoghrajesh@jscheffl