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

New cdd functions #83

Merged
merged 175 commits into from
Oct 30, 2022
Merged

New cdd functions #83

merged 175 commits into from
Oct 30, 2022

Conversation

Brandhoej
Copy link
Contributor

@Brandhoej Brandhoej commented Oct 29, 2022

Merges all the work done on the branch "new-cdd-functions" into "main".

Description

A lot of work has been done on the branch "new-cdd-functions" which should be part of future releases of "main". This PR is a part of the work which aims towards a more structured development (Branch strategy) with shorter branch lifetimes and a specified approach for contribution and collaboration (Inspired by #75).

florber and others added 30 commits June 30, 2022 09:25
@Brandhoej Brandhoej requested a review from magoorden October 29, 2022 08:27
@Brandhoej Brandhoej marked this pull request as ready for review October 29, 2022 08:33
@magoorden
Copy link
Contributor

How should we approach the review of this? It contains 175 commits, so a normal full review is infeasible. Maybe just merge this as-is, so we can continue our development?

@Brandhoej
Copy link
Contributor Author

How should we approach the review of this? It contains 175 commits, so a normal full review is infeasible. Maybe just merge this as-is, so we can continue our development?

@magoorden yes I believe that would be the right choice. Given that all my commits in this branch has already been reviewed (Which is most of them).

@magoorden magoorden merged commit 9d53d7a into main Oct 30, 2022
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.

4 participants