-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Labels
help wantedExtra attention is neededExtra attention is needed
Description
Were added by #83 for public-api rule by @Krakazybik
Should be merge with base config, or stay separated "more lite" version
Why experimental?
Variant 1: public-api/lite (lite)
Without SegmentsAPI/InnerAPI restrictions
// 👍 Pass
/** @path entities/order/index.ts */
import {...} from "./ui";
/** @path features/auth-form/ui/form/content.tsx */
import {...} from "../../model";
// 👍 Also Pass
/** @path widgets/issue-details/index.ts */
import {...} from "./ui/details"
/** @path features/auth-form/index.ts */
import {...} from "./ui/form"
/** @path entities/order/index.ts */
import { saveOrder } from "./model/actions";
/** @path shared/ui */
import { Button } from "./button/button";Variant 2: public-api (base)
With SegmentsAPI/InnerAPI restrictions
// 👍 Pass
/** @path entities/order/index.ts */
import {...} from "./ui";
/** @path features/auth-form/ui/form/content.tsx */
import {...} from "../../model";
// 👎 Fail
/** @path widgets/issue-details/index.ts */
import {...} from "./ui/details"
/** @path features/auth-form/index.ts */
import {...} from "./ui/form"
/** @path entities/order/index.ts */
import { saveOrder } from "./model/actions";
/** @path shared/ui */
import { Button } from "./button/button";Please, leave your vote below:
"👍" - if you prefer to use public-api/lite at base config (less restrictions by default config)
"👎" - if you prefer to use public-api/lite as alternative separated config (more restrictions by default config)
"👀" - if you aren't sure
Feel free leave below your comments
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
help wantedExtra attention is neededExtra attention is needed