Skip to content

Commit 6c7fc50

Browse files
committed
lint fixes
1 parent 27ac5a3 commit 6c7fc50

File tree

6 files changed

+22
-31
lines changed

6 files changed

+22
-31
lines changed

integration-tests/tests/api/organization-access-tokens.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ test.concurrent('create: failure invalid title', async ({ expect }) => {
129129
{
130130
details: {
131131
description: null,
132-
title: Can only contain letters, numbers, " ", '_', and '-'.,
132+
title: Can only contain letters, numbers, " ", "_", and "-".,
133133
},
134134
message: Invalid input provided.,
135135
}

packages/services/api/src/modules/organization/providers/organization-access-tokens.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ import {
3131
const TitleInputModel = z
3232
.string()
3333
.trim()
34-
.regex(/^[ a-zA-Z0-9_-]+$/, `Can only contain letters, numbers, " ", '_', and '-'.`)
34+
.regex(/^[ a-zA-Z0-9_-]+$/, 'Can only contain letters, numbers, " ", "_", and "-".')
3535
.min(2, 'Minimum length is 2 characters.')
3636
.max(100, 'Maximum length is 100 characters.');
3737

packages/web/app/src/components/organization/settings/access-tokens/access-tokens-table.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,8 @@ export function AccessTokensTable(props: AccessTokensTable) {
117117
</Table.TableCell>
118118
<Table.TableCell className="text-right align-middle">
119119
<DropDownMenu.DropdownMenu>
120-
<DropDownMenu.DropdownMenuTrigger className="mr-o ml-auto block">
121-
<EllipsisIcon className="h-4 w-4" />
120+
<DropDownMenu.DropdownMenuTrigger className="ml-auto block">
121+
<EllipsisIcon className="size-4" />
122122
</DropDownMenu.DropdownMenuTrigger>
123123
<DropDownMenu.DropdownMenuContent>
124124
<DropDownMenu.DropdownMenuLabel>Options</DropDownMenu.DropdownMenuLabel>

packages/web/app/src/components/organization/settings/access-tokens/create-access-token-sheet-content.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ import { permissionLevelToResourceName, resolveResources } from './shared-helper
3131
const TitleInputModel = z
3232
.string()
3333
.trim()
34-
.regex(/^[ a-zA-Z0-9_-]+$/, `Can only contain letters, numbers, " ", '_', and '-'.`)
34+
.regex(/^[ a-zA-Z0-9_-]+$/, 'Can only contain letters, numbers, " ", "_", and "-".')
3535
.min(2, 'Minimum length is 2 characters.')
3636
.max(100, 'Maximum length is 100 characters.');
3737

@@ -90,6 +90,7 @@ const CreateAccessTokenSheetContent_CreateOrganizationAccessTokenMutation = grap
9090
export function CreateAccessTokenSheetContent(
9191
props: CreateAccessTokenSheetContentProps,
9292
): React.ReactNode {
93+
// eslint-disable-next-line react/hook-use-state
9394
const [Stepper] = useState(() =>
9495
defineStepper(
9596
{
@@ -359,7 +360,7 @@ export function CreateAccessTokenSheetContent(
359360
<Button
360361
onClick={ev => {
361362
if (stepper.current.id === 'step-1-general') {
362-
Promise.all([form.trigger('title'), form.trigger('description')]).then(
363+
void Promise.all([form.trigger('title'), form.trigger('description')]).then(
363364
([title, description]) => {
364365
if (!title) {
365366
shakeElement(ev);
@@ -377,7 +378,7 @@ export function CreateAccessTokenSheetContent(
377378
}
378379

379380
if (stepper.current.id === 'step-2-permissions') {
380-
form.trigger('permissions').then(permissions => {
381+
void form.trigger('permissions').then(permissions => {
381382
if (!permissions) {
382383
shakeElement(ev);
383384
return;

packages/web/app/src/components/ui/stepper.tsx

Lines changed: 14 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -117,11 +117,12 @@ const defineStepper = <const Steps extends Stepperize.Step[]>(
117117
return (
118118
<nav
119119
aria-label={ariaLabel}
120+
// eslint-disable-next-line jsx-a11y/no-noninteractive-element-to-interactive-role
120121
role="tablist"
121-
className={cn('stepper-navigation', className)}
122+
className={className}
122123
{...props}
123124
>
124-
<ol className={listVariants({ variant: variant })}>{children}</ol>
125+
<ol className={listVariants({ variant })}>{children}</ol>
125126
</nav>
126127
);
127128
},
@@ -150,12 +151,12 @@ const defineStepper = <const Steps extends Stepperize.Step[]>(
150151
return (
151152
<li
152153
className={cn(
153-
'stepper-step flex shrink-0 items-center gap-4 rounded-md transition-colors',
154+
'flex shrink-0 items-center gap-4 rounded-md transition-colors',
154155
className,
155156
)}
156157
>
157158
<CircleStepIndicator currentStep={stepIndex + 1} totalSteps={steps.length} />
158-
<div className="stepper-step-content flex flex-col items-start gap-1">
159+
<div className="flex flex-col items-start gap-1">
159160
{title}
160161
{description}
161162
</div>
@@ -167,7 +168,7 @@ const defineStepper = <const Steps extends Stepperize.Step[]>(
167168
<>
168169
<li
169170
className={cn([
170-
'stepper-step group peer relative flex items-center gap-2',
171+
'group peer relative flex items-center gap-2',
171172
'data-[variant=vertical]:flex-row',
172173
'data-[label-orientation=vertical]:w-full',
173174
'data-[label-orientation=vertical]:flex-col',
@@ -183,10 +184,7 @@ const defineStepper = <const Steps extends Stepperize.Step[]>(
183184
type="button"
184185
role="tab"
185186
tabIndex={dataState !== 'inactive' ? 0 : -1}
186-
className={cn(
187-
'stepper-step-indicator rounded-full',
188-
clickable === false && 'cursor-default',
189-
)}
187+
className={cn('rounded-full', clickable === false && 'cursor-default')}
190188
variant={dataState !== 'inactive' ? 'default' : 'secondary'}
191189
size="icon"
192190
aria-controls={`step-panel-${props.of}`}
@@ -208,7 +206,7 @@ const defineStepper = <const Steps extends Stepperize.Step[]>(
208206
disabled={props.disabled}
209207
/>
210208
)}
211-
<div className="stepper-step-content flex flex-col items-start">
209+
<div className="flex flex-col items-start">
212210
{title}
213211
{description}
214212
</div>
@@ -248,19 +246,15 @@ const defineStepper = <const Steps extends Stepperize.Step[]>(
248246
const { tracking } = useStepperProvider();
249247

250248
return (
251-
<Comp
252-
className={cn('stepper-step-panel', className)}
253-
ref={node => scrollIntoStepperPanel(node, tracking)}
254-
{...props}
255-
>
249+
<Comp className={className} ref={node => scrollIntoStepperPanel(node, tracking)} {...props}>
256250
{children}
257251
</Comp>
258252
);
259253
},
260254
StepperControls: ({ children, className, asChild, ...props }) => {
261255
const Comp = asChild ? Slot : 'div';
262256
return (
263-
<Comp className={cn('stepper-controls flex justify-end gap-4', className)} {...props}>
257+
<Comp className={cn('flex justify-end gap-4', className)} {...props}>
264258
{children}
265259
</Comp>
266260
);
@@ -281,7 +275,7 @@ const StepperTitle = ({
281275
const Comp = asChild ? Slot : 'h4';
282276

283277
return (
284-
<Comp className={cn('stepper-step-title text-base font-medium', className)} {...props}>
278+
<Comp className={cn('text-base font-medium', className)} {...props}>
285279
{children}
286280
</Comp>
287281
);
@@ -300,10 +294,7 @@ const StepperDescription = ({
300294
const Comp = asChild ? Slot : 'p';
301295

302296
return (
303-
<Comp
304-
className={cn('stepper-step-description text-muted-foreground text-sm', className)}
305-
{...props}
306-
>
297+
<Comp className={cn('text-muted-foreground text-sm', className)} {...props}>
307298
{children}
308299
</Comp>
309300
);
@@ -360,7 +351,7 @@ const CircleStepIndicator = ({
360351
aria-valuemin={1}
361352
aria-valuemax={totalSteps}
362353
tabIndex={-1}
363-
className="stepper-step-indicator relative inline-flex items-center justify-center"
354+
className="relative inline-flex items-center justify-center"
364355
>
365356
<svg width={size} height={size}>
366357
<title>Step Indicator</title>
@@ -399,7 +390,7 @@ const CircleStepIndicator = ({
399390

400391
//#region Styles
401392

402-
const listVariants = cva('stepper-navigation-list flex gap-2', {
393+
const listVariants = cva('flex gap-2', {
403394
variants: {
404395
variant: {
405396
horizontal: 'flex-row items-center justify-between',

packages/web/app/src/pages/organization-settings.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ import { Form, FormControl, FormField, FormItem, FormMessage } from '@/component
2828
import { GitHubIcon, SlackIcon } from '@/components/ui/icon';
2929
import { Input } from '@/components/ui/input';
3030
import { Meta } from '@/components/ui/meta';
31-
import { Subtitle, Title } from '@/components/ui/page';
3231
import { NavLayout, PageLayout, PageLayoutContent } from '@/components/ui/page-content-layout';
3332
import { QueryError } from '@/components/ui/query-error';
3433
import { useToast } from '@/components/ui/use-toast';

0 commit comments

Comments
 (0)