Closed PaulWessel closed 3 years ago
Ah, this is what Meghan mentioned yesterday. @Esteban82 does not have the power to approve in this PR... So I need @joa-quim again. We will fix this via the maintainers, writers, admin setup discussed at the meeting.
Paul you don't need our approval. You can merge it without it. I really don't see why we are consuming Watts in CI tests that are testing nothing. This type of changes can be done directly on master.
Paul you don't need our approval. You can merge it without it. I really don't see why we are consuming Watts in CI tests that are testing nothing. This type of changes can be done directly on master.
No, I could not. For some reasons I did not have permission to merge and squash - hence the ping and the discussion yesterday with Meghan about teams... Thanks though!
We can always force merging using administrator rights. There is a button in the PR page saying like "push me to force merging using your administrator powers"
Don't you see this in all PRs?
Not for gmtserver-admin
Paul Wessel, Professor Dept. of Earth Sciences SOEST, U of Hawaii at Manoa
On October 14, 2021 at 11:51:21 AM, Joaquim @.***) wrote:
Don't you see this in all PRs?
[image: image] https://user-images.githubusercontent.com/537321/137400517-7db8aa7c-fe89-4e49-be94-dec04ac1de9a.png
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/GenericMappingTools/gmtserver-admin/pull/120#issuecomment-943763208, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGJ7IX4PX3EDGBBK5GJMCELUG5GFRANCNFSM5GAQSJOQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
Sorry, pinging @joa-quim and @Esteban82 again...