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

Squash post merge commits into one #260

Merged

Conversation

sbidoul
Copy link
Member

@sbidoul sbidoul commented Sep 16, 2023

We squash all post-merge commits done by the bot into one.

This avoid the proliferation of not-very-useful commits that was recently made worse with the README generator that made an additional commit to update the README digest following the version bump.

@sbidoul
Copy link
Member Author

sbidoul commented Sep 16, 2023

I deployed this to test. This seems to work fine. Example here: OCA/purchase-workflow@f78b8fd

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Very needed, yes

@sbidoul
Copy link
Member Author

sbidoul commented Sep 16, 2023

I was sure you would appreciate, @pedrobaeza :)

@sbidoul sbidoul merged commit a90e01a into OCA:master Sep 16, 2023
3 of 5 checks passed
@sbidoul sbidoul deleted the avoid-automated-actions-commits-proliferation branch September 16, 2023 14:56
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.

2 participants