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

Remove mergify config #10068

Merged
merged 2 commits into from
Feb 2, 2023
Merged

Remove mergify config #10068

merged 2 commits into from
Feb 2, 2023

Conversation

tbrlpld
Copy link
Collaborator

@tbrlpld tbrlpld commented Jan 27, 2023

Description

We are not using Mergify at the moment and it keeps throwing errors.
It's also not clear who enabled this in the repo.
There are no details in the password manager.

I hope that removing the config will disable Merify.

Related issue: #10096

Checklist

Tests

  • [ ] Is the code I'm adding covered by tests?

Changes in Models:

  • [ ] Did I update or add new fake data?
  • [ ] Did I squash my migration?
  • [ ] Are my changes backward-compatible. If not, did I schedule a deploy with the rest of the team?

Documentation:

  • Is my code documented?
  • Did I update the READMEs or wagtail documentation?

@tbrlpld tbrlpld force-pushed the remove-mergify-config branch from 0ea36ef to 89f9344 Compare February 2, 2023 19:20
@tbrlpld tbrlpld marked this pull request as ready for review February 2, 2023 19:20
We are not using Mergify at the moment and it keeps throwing errors.
@tbrlpld tbrlpld force-pushed the remove-mergify-config branch from 89f9344 to 3947309 Compare February 2, 2023 19:21
@tbrlpld tbrlpld requested a review from mmmavis February 2, 2023 19:24
Copy link
Collaborator

@mmmavis mmmavis left a comment

Choose a reason for hiding this comment

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

Hopefully this is THE FIX!

@tbrlpld tbrlpld merged commit 7750fb9 into main Feb 2, 2023
@tbrlpld tbrlpld deleted the remove-mergify-config branch February 2, 2023 22:01
@tbrlpld
Copy link
Collaborator Author

tbrlpld commented Feb 2, 2023

🤞

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