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.

Sign up for GitHub

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

Jump to bottom

👷 CI/CD - Make commit history update optional #1365

Merged
merged 8 commits into from
Jun 10, 2024
Merged

👷 CI/CD - Make commit history update optional #1365

merged 8 commits into from
Jun 10, 2024

Conversation

bradystroud
Copy link
Member

@bradystroud bradystroud commented Jun 8, 2024 •
edited
Loading

Relates to #1367

Sometimes there are huge automated changes to lots of rules, we dont want these in the history.
This option allows them to be skipped.

@bradystroud bradystroud enabled auto-merge (squash) June 8, 2024 02:34
@bradystroud bradystroud changed the title CI/CD - Make commit history optional 🔧 CI/CD - Make commit history optional Jun 8, 2024
@bradystroud bradystroud changed the title 🔧 CI/CD - Make commit history optional 👷 CI/CD - Make commit history optional Jun 8, 2024
@bradystroud
Copy link
Member Author

bradystroud commented Jun 8, 2024 •
edited
Loading

Closing as I have decided to go with another approach.
I think this is still good

See #1307

@bradystroud bradystroud closed this Jun 8, 2024
auto-merge was automatically disabled June 8, 2024 03:21

Pull request was closed

@bradystroud bradystroud mentioned this pull request Jun 8, 2024
Closed
@bradystroud bradystroud reopened this Jun 10, 2024
@bradystroud bradystroud changed the title 👷 CI/CD - Make commit history optional 👷 CI/CD - Make commit history update optional Jun 10, 2024
JackDevAU
JackDevAU approved these changes Jun 10, 2024
View reviewed changes
Copy link
Member

@JackDevAU JackDevAU left a comment

Choose a reason for hiding this comment

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

🚀

JackDevAU
JackDevAU approved these changes Jun 10, 2024
View reviewed changes
Copy link
Member

@JackDevAU JackDevAU left a comment

Choose a reason for hiding this comment

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

🚀

@bradystroud bradystroud enabled auto-merge (squash) June 10, 2024 02:54
@bradystroud bradystroud mentioned this pull request Jun 10, 2024
Closed
@bradystroud bradystroud merged commit ce4d079 into main Jun 10, 2024
10 checks passed
@bradystroud bradystroud deleted the optional-commit-history branch June 10, 2024 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JackDevAU JackDevAU JackDevAU approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bradystroud @JackDevAU