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

Get the final status of moderated subjects (also filtering subsets). #7732

Closed
wants to merge 1 commit into from

Conversation

isoos
Copy link
Collaborator

@isoos isoos commented May 15, 2024

@isoos isoos requested a review from jonasfj May 15, 2024 12:58
/// Returns the list of subjects that have a final state as moderated.
/// If a package version and also a package is moderated, returns
/// only the package.
List<String> moderatedSubjects() {
Copy link
Member

Choose a reason for hiding this comment

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

Find state is difficult. Let's not try to do that.
Let's just return the log and let humans read it?

Copy link
Member

Choose a reason for hiding this comment

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

I mean things could happen in other cases concurrently.

@isoos isoos closed this May 17, 2024
@isoos isoos deleted the case-log branch May 17, 2024 14:44
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