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

fix SEX to SDV #34

Merged
merged 1 commit into from
Jun 6, 2023
Merged

fix SEX to SDV #34

merged 1 commit into from
Jun 6, 2023

Conversation

JohnMoehrke
Copy link
Contributor

close #33

fix SEX to SDV
@JohnMoehrke JohnMoehrke requested a review from mojitoj June 6, 2023 14:27
Copy link
Contributor

@mojitoj mojitoj left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@JohnMoehrke
Copy link
Contributor Author

let me ask Luke to approve and see if that works.

@mojitoj
Copy link
Contributor

mojitoj commented Jun 6, 2023 via email

@JohnMoehrke
Copy link
Contributor Author

@lukeaduncan strange branch control is happening. I requested Mohammad to approve, he did, but that didn't unblock the pull-request. Given that this is the new branch control we are using in IHE, I want to figure out what is not working. So please approve, and we will see if YOUR permissions unblocks the pull request.

@JohnMoehrke
Copy link
Contributor Author

note.. I know that I can override the block... but want to figure out the right thing as not all committees have a user with the authority that I have.

Copy link
Contributor

@lukeaduncan lukeaduncan left a comment

Choose a reason for hiding this comment

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

Looks good.

@JohnMoehrke
Copy link
Contributor Author

so, somehow luke is more powerful than Mohammad.

@lukeaduncan
Copy link
Contributor

note.. I know that I can override the block... but want to figure out the right thing as not all committees have a user with the authority that I have.

It looks like it was saying it needed a review by someone with write access. I don't know if that was a difference or not.

@mojitoj
Copy link
Contributor

mojitoj commented Jun 6, 2023 via email

@JohnMoehrke
Copy link
Contributor Author

I ddin't see in the branch control where it requires a review of someone with write... that might be a problem with our plan on branch control. or do we want co-chairs to approve everything?

@JohnMoehrke JohnMoehrke merged commit 4ddc1d6 into master Jun 6, 2023
@JohnMoehrke JohnMoehrke deleted the SDV branch June 6, 2023 15:57
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.

SEX is used with SDV definition
3 participants