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

DDS: honor ImageInput thread policy #3584

Merged
merged 1 commit into from
Oct 6, 2022

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Oct 6, 2022

No description provided.

@lgritz
Copy link
Collaborator Author

lgritz commented Oct 6, 2022

@aras-p

@aras-p
Copy link
Contributor

aras-p commented Oct 6, 2022

Github UI does not allow me to set this as "reviewed", so I'll just write 👍 🟢 here.

@lgritz
Copy link
Collaborator Author

lgritz commented Oct 6, 2022

GH only lets you designate somebody as a reviewer if they are an "org member", which I think means a designated committer with write perms. That's crazy, it comes up all the time that you might want a review by the person who reported a problem, or an expert in some area who is not a frequent contributor, etc. I understand why you don't want to allow any repo to request a review from any user at all -- it would be a way for people to spam well-known developers -- but I don't see why it wouldn't be simple to allow anyone who has previously interacted with the project to have their review requested or accepted.

@lgritz lgritz merged commit b89fa9d into AcademySoftwareFoundation:master Oct 6, 2022
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Oct 6, 2022
@lgritz lgritz deleted the lg-ddspar branch October 6, 2022 06:58
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.

None yet

2 participants