FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix constructors with configuration by MichelZ · Pull Request #133 · openmcdf/openmcdf · GitHub

Fix constructors with configuration - #133

Merged
ironfede merged 1 commit into
openmcdf:masterfrom
MichelZ:mize/fix_constructors_with_configuration
Aug 3, 2024
Merged

ironfede merged 1 commit into
openmcdf:masterfrom
MichelZ:mize/fix_constructors_with_configuration

Conversation

MichelZ commented May 31, 2024

Copy link
Copy Markdown
Contributor
  • There are issues with certain constructors that not all configuration options are set properly. This is now consolidated to one function
  • Constructors also refer to each other when possible to not duplicate too much code

ironfede commented Aug 3, 2024

Copy link
Copy Markdown
Collaborator

Thank you @MichelZ for improvements

ironfede merged commit 7926818 into openmcdf:master Aug 3, 2024
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL