Revert "Develop into main"#58
Merged
davidirvine merged 1 commit intomainfrom Jul 25, 2025
Merged
Conversation
This reverts commit ea0dd8c.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request reverts a previous merge commit that added comprehensive testing infrastructure and documentation to the deepnote project. The revert removes extensive test suites, performance benchmarks, CI/CD workflows, documentation generation setup, and development tooling, simplifying the codebase back to its core functionality.
- Removes comprehensive testing infrastructure including property-based tests, stress tests, and performance benchmarks
- Eliminates CI/CD pipeline with automated testing, code quality checks, and documentation deployment
- Strips away extensive documentation including API docs, performance guides, and usage examples
- Simplifies code formatting by removing detailed style configurations and static analysis tools
Reviewed Changes
Copilot reviewed 26 out of 32 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| test/voice.cpp | Reformats function definitions to use multi-line style instead of single-line |
| test/statemachine_lifecycle_tests.cpp | Completely removes comprehensive state machine validation tests |
| test/property_edge_tests.cpp | Removes property-based testing and edge case validation |
| test/performance_tests.cpp | Eliminates real-time performance validation and benchmarking |
| test/freqtable.cpp | Minor formatting changes to function definitions and array initialization |
| test/error_robustness_tests.cpp | Removes comprehensive error handling and robustness validation |
| test/bezier_animation_tests.cpp | Eliminates Bezier curve mathematical property validation tests |
| test/CMakeLists.txt | Removes compiler optimization flags and simplifies build configuration |
| src/voice/frequencytable.hpp | Minor formatting and style changes to class definitions |
| src/voice/deepnotevoice.hpp | Significant refactoring from array-based to vector-based oscillator storage |
| src/util/namedtype.hpp | Formatting changes to constructor definitions |
| src/unitshapers/linear.hpp | Minor formatting adjustment to function definition |
| src/unitshapers/bezier.hpp | Removes extensive documentation comments and reformats constructor |
| src/ranges/scaler.hpp | Formatting changes to constructor definitions |
| src/ranges/range.hpp | Formatting adjustments to constructor and method definitions |
Comments suppressed due to low confidence (1)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reverts #51