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

Add GitHub action to check for merge conflicts #15673

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

barrettj12
Copy link
Contributor

When a PR is merged, check if it creates merge conflicts. If so, notify the PR author on Mattermost.

@hpidcock hpidcock added the 2.9 label May 30, 2023
@barrettj12 barrettj12 marked this pull request as ready for review May 30, 2023 03:09
@barrettj12

This comment was marked as outdated.

@barrettj12 barrettj12 requested a review from hpidcock May 30, 2023 23:21
@barrettj12
Copy link
Contributor Author

Tested the workflow on my fork here: https://github.com/barrettj12/juju/actions/runs/5138690057/jobs/9248283137

Copy link
Member

@hpidcock hpidcock left a comment

Choose a reason for hiding this comment

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

Cool, thanks this will hopefully improve things.

@barrettj12
Copy link
Contributor Author

/merge

@jujubot jujubot merged commit 5c253c3 into juju:2.9 Jun 1, 2023
3 checks passed
@barrettj12 barrettj12 deleted the check-merge branch June 1, 2023 01:30
@hpidcock hpidcock mentioned this pull request Jun 7, 2023
jujubot added a commit that referenced this pull request Jun 7, 2023
#15710

Forward ports:
- #15676
- #15672
- #15683
- #15673
- #15677
- #15691
- #15701

Conflicts:
- apiserver/errors/errors.go
- apiserver/facades/client/application/application.go
- apiserver/facades/client/application/application_unit_test.go
- apiserver/restrict_newer_client_test.go
- go.mod
- go.sum
- scripts/win-installer/setup.iss
- snap/snapcraft.yaml
- state/migration_import_test.go
- state/mocks/description_mock.go
- version/version.go
@wallyworld wallyworld mentioned this pull request Jun 8, 2023
jujubot added a commit that referenced this pull request Jun 8, 2023
#15717

Merge 3.1

#15676 [from wallyworld/newer-clients-migrate](7c1f884)
#15672 [from hpidcock/bump-juju-description-v3.0.15](acec126)
#15683 [from hpidcock/fix-persistent-storage-test](460dd21)
#15673 [from barrettj12/check-merge](5c253c3)
#15677 [from barrettj12/invalid-offer](3cb3f8b)
#15654 [from ycliuhw/fix/backendRefCount](4e5ae3c)
#15692 [from wallyworld/fix-secrets-cmr](a1fb0c4)
[t](840bc09) #15709 [from wallyworld/hook-secret-revison](840bc09)
#15701 [from hpidcock/fix-upgrade-podspec-sidecar](d465c93)
#15681 [from anvial/JUJU-3882-fix-test-deploy-test-…](f54c1ad)
#15714 [from wallyworld/offer-consume=sameapp](6390036)

Conflicts were upgrade steps - the 3.1.3 step has been moved to 3.2.1.
Also an auth tweak to crossmodelrelaltions.

```
# Conflicts:
# apiserver/common/crossmodel/auth_test.go
# apiserver/facades/controller/crossmodelrelations/crossmodelrelations.go
# rpc/params/apierror.go
# state/upgrades.go
# state/upgrades_test.go
# upgrades/backend.go
#
```

## QA steps

See PRs
This was referenced Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants