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

Fix filename clash detection with virtual files #8339

Merged
merged 3 commits into from
Jan 19, 2021

Conversation

TheOneRing
Copy link
Member

@TheOneRing TheOneRing commented Jan 5, 2021

Bugfix: Detect file name clash with VirtualFiles enabled

We fixed an issue where the file name clash detection was not run with
VirtualFiles enabled.

#8323

@TheOneRing TheOneRing force-pushed the work/cleanup_localFileNameClash branch from 2055e3b to 825f80c Compare January 5, 2021 10:17
@TheOneRing TheOneRing force-pushed the work/cleanup_localFileNameClash branch 2 times, most recently from 53143cd to 55b44be Compare January 13, 2021 13:38
@TheOneRing TheOneRing changed the title Cleanup localFileNameClash Fix filename clash detection Jan 13, 2021
@TheOneRing TheOneRing changed the title Fix filename clash detection Fix filename clash detection with virtual files Jan 13, 2021
@sonarcloud
Copy link

sonarcloud bot commented Jan 13, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Contributor

@dragotin dragotin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Even my name mentioned ;-)

qCWarning(lcPropagator) << "No valid fileinfo";
return false;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not like this style. Tbh.

@TheOneRing TheOneRing changed the base branch from 2.7 to work/v2.7.5 January 18, 2021 21:49
@TheOneRing
Copy link
Member Author

TODO @TheOneRing rebase on release branch...

@TheOneRing TheOneRing force-pushed the work/cleanup_localFileNameClash branch from 55b44be to 7110c09 Compare January 19, 2021 07:58
@TheOneRing TheOneRing merged commit e25f1b2 into work/v2.7.5 Jan 19, 2021
@delete-merged-branch delete-merged-branch bot deleted the work/cleanup_localFileNameClash branch January 19, 2021 07:59
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.

None yet

2 participants