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

Build: Update aqt to 2.2.3 (Automated PR) #2787

Closed
wants to merge 1 commit into from

Conversation

github-actions[bot]
Copy link

This automated Pull Request updates the used aqt version to version 2.2.3.

This PR was opened by the workflow Bump dependencies (create-prs)

CHANGELOG: SKIP

Copy link
Member

@hoffie hoffie left a comment

Choose a reason for hiding this comment

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

@hoffie hoffie added this to the Release 3.9.1 milestone Aug 19, 2022
@hoffie hoffie force-pushed the ci/bump-dependencies/aqt branch 2 times, most recently from 3819d76 to bde865b Compare August 22, 2022 17:59
@ann0see
Copy link
Member

ann0see commented Aug 22, 2022

@hoffie I think the name of the commit and depandabot should be the same "CI: Bump x from a to b"

@hoffie hoffie force-pushed the ci/bump-dependencies/aqt branch 7 times, most recently from 15a9483 to 335cd7c Compare August 22, 2022 20:32
@ann0see
Copy link
Member

ann0see commented Aug 22, 2022

Please ping me if you're finished.

Comment on lines 140 to 147
# Use Github's API directly instead of using `gh edit`.
# The latter internally queries all fields of the PR which leads to
# a permission error as the workflow does not have access to
# organization projects:
gh api --silent -H "Accept: application/vnd.github+json" --method PATCH \
"/repos/${GITHUB_REPOSITORY}/pulls/${existing_pr}" \
-f title="${pr_title}" \
-f body="${body}"
Copy link
Member

Choose a reason for hiding this comment

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

This appears as part of this PR as the last workflow run was from the PR branch for #2801. It will be fixed once that PR is merged and the workflow runs again.

@hoffie hoffie marked this pull request as draft August 22, 2022 21:19
@hoffie hoffie force-pushed the ci/bump-dependencies/aqt branch 3 times, most recently from c4a406d to 9be9714 Compare August 23, 2022 20:09
@hoffie
Copy link
Member

hoffie commented Aug 25, 2022

(Keeping still as draft as we have to approve/merge #2801 first).

@hoffie hoffie force-pushed the ci/bump-dependencies/aqt branch 2 times, most recently from b3b649d to 1d64f61 Compare August 27, 2022 00:00
@hoffie hoffie closed this in #2801 Aug 28, 2022
@github-actions github-actions bot deleted the ci/bump-dependencies/aqt branch August 28, 2022 19:16
@hoffie
Copy link
Member

hoffie commented Aug 28, 2022

This was incorrectly closed by #2801, because I had added "Fixes: ... url to a comment in this PR...". Usually, the workflow should have updated this PR instead of creating a new one (#2814).

ann0see pushed a commit to ann0see/jamulus that referenced this pull request Nov 9, 2022
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.

4 participants