Skip to content

Comments

Folder Refactor#158

Open
c-hayCodes wants to merge 5 commits intoReduxISU:CSharpAPIfrom
c-hayCodes:CSharpAPIFolderRefactor
Open

Folder Refactor#158
c-hayCodes wants to merge 5 commits intoReduxISU:CSharpAPIfrom
c-hayCodes:CSharpAPIFolderRefactor

Conversation

@c-hayCodes
Copy link
Contributor

refactored the Navigation Problems controllers, allowing for problem to be placed into multiple different folders based on class (P, NP, NPHard)

@wrigjl
Copy link
Collaborator

wrigjl commented Feb 13, 2026

I don't know how this happened, but you've managed to revert changes to bernstein vazarani. Please back out your changes to both of the files below:

Problems/NPComplete/NPC_BERNSTEINVAZIRANI/BERNSTEINVAZIRANI_Class.cs
redux-tests/Problems/NPC_BERNSTEINVAZIRANI/BERNSTEINVAZIRANI_Tests.cs

You are proposing to undo changes I intended for this problem and tests (updates to the documentation and to the representation of the problem).

My guess is that when you merged with main you dealt with the conflict by accepting only your changes and thus reverted the changes I made. You didn't have conflicts in the tests because you didn't modify them.

@wrigjl
Copy link
Collaborator

wrigjl commented Feb 13, 2026

My name is Jason Wright and I approve of this diff =)

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.

2 participants