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

When autocomplete is used, the PR number cannot be retrieved #54

Open
vmartinlantek opened this issue May 22, 2023 · 1 comment · May be fixed by #71
Open

When autocomplete is used, the PR number cannot be retrieved #54

vmartinlantek opened this issue May 22, 2023 · 1 comment · May be fixed by #71

Comments

@vmartinlantek
Copy link

vmartinlantek commented May 22, 2023

When autocomplete is used, the PR merged message changes (no longer starts with Merged PR) and it is not possible to retrieve the PR number

def get_pr_num(self, commit_id) -> str:
        comment = self.get_commit_message(commit_id)
        MERGED_PR = "Merged PR "

The actual message is

"comment": "Merge pull request XXXX from dev/YYYY/zzzzz into main",
@vmartinlantek vmartinlantek changed the title When autocomplete is used the PR cannot be retrieved When autocomplete is used, the PR number cannot be retrieved May 25, 2023
@pareion
Copy link
Contributor

pareion commented Jan 16, 2024

I had the same issue as you; luckily for me, I set the autocomplete through the Azure CLI command to update the pull request.

There is a parameter you can supply called --merge-commit-message, which you can add Merged PR to.

az repos pr update --id ${{ parameters.pullRequestId }} --auto-complete --org "${{ parameters.organization }}" --merge-commit-message "Merged PR ${{ parameters.pullRequestId }}: $title".

However, I haven't found a way of handling it through the UI in Azure DevOps..

@bonddim bonddim linked a pull request Jun 28, 2024 that will close this issue
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 a pull request may close this issue.

2 participants