-
-
Notifications
You must be signed in to change notification settings - Fork 670
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
fix: ignore this #2834
fix: ignore this #2834
Conversation
This is a dummy commit to create a new release.
Do you know why our release pipeline does not work? |
All reactions
Sorry, something went wrong.
I found the reason, aspublish requires |
All reactions
Sorry, something went wrong.
@HerrCai0907 That would be nice, but if a PR only has one commit, the default commit title in the merge commit is based off of that commit's title, not the PR title (see #2826). |
All reactions
Sorry, something went wrong.
Yes. But we can switch the merge strategy in configuration page. I am not sure who has the access to setup it. Maybe it needs @dcodeIO or @MaxGraey |
All reactions
Sorry, something went wrong.
@HerrCai0907 that doesn't sound ideal... |
All reactions
Sorry, something went wrong.
Sorry, something went wrong.
Changed the setting to "Pull request title" as suggested |
All reactions
-
👍 1 reaction
Sorry, something went wrong.
@HerrCai0907 Oh I misunderstood (I thought you meant switching away from squash merging). That's perfect! |
All reactions
Sorry, something went wrong.
This is a dummy commit to create a new release.