[fix] Check styles that was imported from other file

Intellicode / eslint-plugin-react-native

React Native plugin for ESLint

MIT License

722 stars 130 forks source link

[fix] Check styles that was imported from other file #303

Closed retyui closed 1 year ago

retyui commented 2 years ago

Issue: https://github.com/Intellicode/eslint-plugin-react-native/issues/165#issuecomment-983636750

retyui commented 2 years ago

Demo

https://user-images.githubusercontent.com/4661784/144732862-17b64c75-260b-48a4-a0b3-fd50c9fbf63c.mp4

retyui commented 2 years ago

@Intellicode I see that all last pipelines on a master branch are red failed.

For me as a contributor is important that nothing was broken when I made changes

Intellicode commented 2 years ago

@Intellicode I see that all last pipelines on a master branch are red failed.

For me as a contributor is important that nothing was broken when I made changes

@retyui, I agree and I understand, unfortunately the effort required to make all (updated) dependencies compatible with each other is more than I wished. Hope to have a working build soon.

Thanks for your PR!

Intellicode commented 2 years ago

Feel free to re-integrate the main branch, I think it's fixed now!

retyui commented 2 years ago

@Intellicode so fast 👍 that's incredible!! I will check my PR now

retyui commented 2 years ago

@Intellicode I've just added tests for new code, and found & fixed a small bug)

Azure Pipelines now green!
But SonarCloud failed, and want's to fix 21.4% Duplication

I don't know how to mark no-unused-styles.experimental.js as excluded file to calculate the duplication

you can review PR @Intellicode

Intellicode commented 2 years ago

@retyui Thanks for updating! Please allow some time for a review, since it is a big PR. It looks very nice and useful though!

retyui commented 2 years ago

@Intellicode did you have time to check PR?

Intellicode commented 2 years ago

@retyui, sorry not yet! I'm afraid it will be after Christmas due to my availability.

retyui commented 2 years ago

@Intellicode another polite reminder

Intellicode commented 2 years ago

@Intellicode another polite reminder

Sorry, I didn't find availability yet to have a proper look, unfortunately I'm unable to rush it, since this PR introduces two new dependencies ("eslint-import-resolver-node": "^0.3.6", "eslint-module-utils": "^2.7.1"), I will have to vet them as well, that will take more time.