Skip to content

Commit 39ea7d4

Browse files
committed
fix(onboarding): Fix knip and typecheck CI failures
Un-export OnboardingSessionState (no longer imported externally). Add @public JSDoc tags to SCM prop interfaces so knip recognizes them as intentionally exported for the project creation flow. Fix renderComponent type in scmPlatformFeatures.spec.tsx. ref(onboarding): Un-export SCM prop interfaces until needed No external consumers yet. Re-export when the project creation flow imports them.
1 parent 7a697e6 commit 39ea7d4

File tree

6 files changed

+6
-6
lines changed

6 files changed

+6
-6
lines changed

static/app/components/onboarding/onboardingContext.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ type OnboardingContextProps = {
1919
selectedRepository?: Repository;
2020
};
2121

22-
export type OnboardingSessionState = {
22+
type OnboardingSessionState = {
2323
createdProjectSlug?: string;
2424
selectedFeatures?: ProductSolution[];
2525
selectedIntegration?: Integration;

static/app/views/onboarding/components/scmRepoSelector.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import {ScmVirtualizedMenuList} from './scmVirtualizedMenuList';
1212
import {useScmRepos} from './useScmRepos';
1313
import {useScmRepoSelection} from './useScmRepoSelection';
1414

15-
export interface ScmRepoSelectorProps {
15+
interface ScmRepoSelectorProps {
1616
integration: Integration;
1717
onRepositoryChange: (repo: Repository | undefined) => void;
1818
selectedRepository: Repository | undefined;

static/app/views/onboarding/scmConnect.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ import {useScmPlatformDetection} from './components/useScmPlatformDetection';
2222
import {useScmProviders} from './components/useScmProviders';
2323
import {SCM_STEP_CONTENT_WIDTH} from './consts';
2424

25-
export interface ScmConnectProps {
25+
interface ScmConnectProps {
2626
onComplete: () => void;
2727
onIntegrationChange: (integration: Integration | undefined) => void;
2828
onRepositoryChange: (repo: Repository | undefined) => void;

static/app/views/onboarding/scmPlatformFeatures.spec.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ describe('ScmPlatformFeatures', () => {
6767

6868
function renderComponent(
6969
overrides?: Partial<typeof defaultProps>,
70-
orgOverride?: Parameters<typeof render>[1]['organization']
70+
orgOverride?: typeof organization
7171
) {
7272
const props = {...defaultProps, ...overrides};
7373
return render(<ScmPlatformFeatures {...props} />, {

static/app/views/onboarding/scmPlatformFeatures.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ function shouldSuggestFramework(platformKey: PlatformKey): boolean {
7575
// Wider than SCM_STEP_CONTENT_WIDTH (506px) used by the footer.
7676
const PLATFORM_CONTENT_WIDTH = '564px';
7777

78-
export interface ScmPlatformFeaturesProps {
78+
interface ScmPlatformFeaturesProps {
7979
onComplete: () => void;
8080
onFeaturesChange: (features: ProductSolution[]) => void;
8181
onPlatformChange: (platform: OnboardingSelectedSDK | undefined) => void;

static/app/views/onboarding/scmProjectDetails.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ import {ScmStepHeader} from './components/scmStepHeader';
3131

3232
const PROJECT_DETAILS_WIDTH = '285px';
3333

34-
export interface ScmProjectDetailsProps {
34+
interface ScmProjectDetailsProps {
3535
onComplete: (
3636
platform?: OnboardingSelectedSDK,
3737
query?: Record<string, string[]>

0 commit comments

Comments
 (0)