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.

Sign up for GitHub

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

Jump to bottom

Fix missing graphviz config breaks command usage #951

Merged
merged 1 commit into from
Aug 12, 2022
Merged

Fix missing graphviz config breaks command usage #951

merged 1 commit into from
Aug 12, 2022

Conversation

dbrumann
Copy link
Collaborator

No description provided.

@dbrumann dbrumann merged commit c69ee36 into qossmic:main Aug 12, 2022
@dbrumann dbrumann deleted the issue-949 branch August 12, 2022 09:16
This was referenced Aug 12, 2022
Closed
Closed
xabbuh added a commit that referenced this pull request Jul 21, 2023
@xabbuh
do not crash the GraphViz formatter if no config is given
06642bd 
PR #951 did not fully fix the issue. Reason for this is that the Symfony
container compiler skips loading extensions for which no configuration has
been made. This in turn means that the Deptrac Configuration class is never
processed and its default values are not being populated.
@xabbuh xabbuh mentioned this pull request Jul 21, 2023
Merged
xabbuh added a commit that referenced this pull request Jul 21, 2023
@xabbuh
do not crash the GraphViz formatter if no config is given
8215b12 
PR #951 did not fully fix #949. Reason for this is that the Symfony container
compiler skips loading extensions for which no configuration has been made.
This in turn means that the Deptrac Configuration class is never processed
and its default values are not being populated.
gennadigennadigennadi pushed a commit that referenced this pull request Jul 21, 2023
@xabbuh
do not crash the GraphViz formatter if no config is given (#1251)
40d6091 
PR #951 did not fully fix #949. Reason for this is that the Symfony container
compiler skips loading extensions for which no configuration has been made.
This in turn means that the Deptrac Configuration class is never processed
and its default values are not being populated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dbrumann