Skip to content
This repository has been archived by the owner on Dec 6, 2022. It is now read-only.

Better use namespace export l1 #53

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Better use namespace export l1 #53

wants to merge 21 commits into from

Conversation

rMazeiks
Copy link
Contributor

  • Delete old docs pages (for rules that no longer exist
  • export-level-one and use-namespace (the most commonly encountered rules):
    • Add tests
    • Update docs (for derived-class-names too)
    • Improve error messages
    • Minor refactoring

Copy link
Contributor

@yeegor yeegor left a comment

Choose a reason for hiding this comment

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

This will get merged after Mosaic/CSA merge, after corresponding changes have been implemented.

Comment on lines 9 to 13
// in Goodbye.component.js
class HelloComponent {}

// in Hello.component.js
class HelloComponent { /** ... */ }
// in Footer.container.js
class FooterComponent {}
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add example without prefix at all (e.g. Hello instead of HelloComponent).
Also, please reference that ScandiPWA core is not compliant with this rule as of now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the feedback; fixed!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants