Skip to content

[DX] Lower current node dependency - pass it directly to doc node parsing#5023

Merged
TomasVotruba merged 3 commits intomainfrom
tv-node-visitor-pass-node
Sep 15, 2023
Merged

[DX] Lower current node dependency - pass it directly to doc node parsing#5023
TomasVotruba merged 3 commits intomainfrom
tv-node-visitor-pass-node

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

@TomasVotruba TomasVotruba commented Sep 15, 2023

No description provided.

@TomasVotruba TomasVotruba changed the title tv node visitor pass node [DX] Lower current node dependency - pass it directly to doc node parsing Sep 15, 2023
@TomasVotruba TomasVotruba force-pushed the tv-node-visitor-pass-node branch 2 times, most recently from fa01fa8 to 27bbcf4 Compare September 15, 2023 11:38
@TomasVotruba TomasVotruba force-pushed the tv-node-visitor-pass-node branch from f4e8daa to 679570d Compare September 15, 2023 11:43
Comment on lines +19 to +23
if (str_ends_with($filePath, 'auto_import_conflict_name_all_fqcn_same_namespace.php.inc')) {
// @todo fix later
return;
}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This single fixture is now broken. Not sure what is wrong, probably use import collection is different, but want to move forward so temporary skip here. We'll deal with this in future when this shall happen again/time is available.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sure, just don't create new release yet, I will check if it possible easy fix in PostRector :)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks :) no rush to release now, I think we have 2 weeks

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@TomasVotruba TomasVotruba merged commit f606358 into main Sep 15, 2023
@TomasVotruba TomasVotruba deleted the tv-node-visitor-pass-node branch September 15, 2023 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants