Skip to content

Commit 576c4c2

Browse files
committed
Fix linting issues
1 parent 3eaa1f4 commit 576c4c2

File tree

8 files changed

+31
-18
lines changed

8 files changed

+31
-18
lines changed

src/components/checkboxes/Checkboxes.tsx

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,13 @@ class Checkboxes extends PureComponent<CheckboxesProps, CheckboxesState> {
8181
const { children, ...rest } = this.props;
8282
return (
8383
<FormGroup<CheckboxesProps> inputType="checkboxes" {...rest}>
84-
{/* eslint-disable-next-line @typescript-eslint/no-unused-vars */}
8584
{({
86-
className, name, id, idPrefix, ...restRenderProps
85+
className,
86+
name,
87+
id,
88+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
89+
idPrefix,
90+
...restRenderProps
8791
}) => {
8892
this.resetBoxIds();
8993
const containsConditional = this.state.conditionalBoxes.length > 0;

src/components/date-input/DateInput.tsx

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ type DateInputChangeEvent = ChangeEvent<HTMLInputElement> & {
1818

1919
interface DateInputProps
2020
extends Omit<HTMLProps<HTMLDivElement>, 'value' | 'defaultValue'>,
21-
FormElementProps {
21+
FormElementProps {
2222
autoSelectNext?: boolean;
2323
value?: Partial<DateInputValue>;
2424
defaultValue?: Partial<DateInputValue>;
@@ -118,17 +118,19 @@ class DateInput extends PureComponent<DateInputProps, DateInputState> {
118118
};
119119

120120
render(): JSX.Element {
121-
// eslint-disable-next-line @typescript-eslint/no-unused-vars
122121
const {
123-
children, onChange, value, defaultValue, ...rest
122+
children,
123+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
124+
onChange,
125+
value,
126+
defaultValue,
127+
...rest
124128
} = this.props;
125129

126130
return (
127131
<FormGroup<Omit<DateInputProps, 'value' | 'defaultValue'>> inputType="dateinput" {...rest}>
128132
{/* eslint-disable-next-line @typescript-eslint/no-unused-vars */}
129-
{({
130-
className, name, id, error, autoSelectNext, ...restRenderProps
131-
}) => {
133+
{({ className, name, id, error, autoSelectNext, ...restRenderProps }) => {
132134
const contextValue: IDateInputContext = {
133135
id,
134136
name,

src/components/radios/Radios.tsx

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,10 @@ class Radios extends PureComponent<RadiosProps, RadiosState> {
2424

2525
private radioIds: Record<string, string> = {};
2626

27+
static Divider = Divider;
28+
29+
static Radio = Radio;
30+
2731
static defaultProps = {
2832
role: 'radiogroup',
2933
};
@@ -89,14 +93,12 @@ class Radios extends PureComponent<RadiosProps, RadiosState> {
8993
this.radioIds = {};
9094
};
9195

92-
render() {
96+
render(): JSX.Element {
9397
const { children, ...rest } = this.props;
9498
return (
9599
<FormGroup<RadiosProps> inputType="radios" {...rest}>
96100
{/* eslint-disable-next-line @typescript-eslint/no-unused-vars */}
97-
{({
98-
className, inline, name, id, error, ...restRenderProps
99-
}) => {
101+
{({ className, inline, name, id, error, ...restRenderProps }) => {
100102
this.resetRadioIds();
101103
const contextValue: IRadiosContext = {
102104
getRadioId: (reference) => this.getRadioId(id, reference),

src/components/skip-link/SkipLink.tsx

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,9 +85,13 @@ class SkipLink extends React.Component<SkipLinkProps> {
8585
};
8686

8787
render(): JSX.Element {
88-
// eslint-disable-next-line @typescript-eslint/no-unused-vars
8988
const {
90-
className, focusTargetRef, disableDefaultBehaviour, href, ...rest
89+
className,
90+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
91+
focusTargetRef,
92+
disableDefaultBehaviour,
93+
href,
94+
...rest
9195
} = this.props;
9296
return (
9397
<a

src/components/table/Table.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ class Table extends React.PureComponent<TableProps, TableState> {
4343
};
4444
}
4545

46-
setHeadings = (headings: string[]) => {
46+
setHeadings = (headings: string[]): void => {
4747
const isEqual = headings.reduce(
4848
(prevValue, heading, index) => prevValue && heading === this.state.headings[index],
4949
true,
@@ -52,7 +52,7 @@ class Table extends React.PureComponent<TableProps, TableState> {
5252
if (!isEqual) this.setState({ headings });
5353
};
5454

55-
render() {
55+
render(): JSX.Element {
5656
const { className, responsive, children, caption, captionProps, ...rest } = this.props;
5757

5858
const contextValue: ITableContext = {

src/components/table/TableContext.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ export interface ITableContext {
77
}
88

99
const TableContext = createContext<ITableContext>({
10+
/* eslint-disable @typescript-eslint/no-empty-function */
1011
isResponsive: false,
1112
headings: [],
1213
setHeadings: () => {},

src/util/hooks/UseDevWarning.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import isDev from '../IsDev';
33

44
type ConditionFn = () => boolean;
55

6-
const useDevWarning = (warning: string, condition: ConditionFn = () => true) => {
6+
const useDevWarning = (warning: string, condition: ConditionFn = () => true): void => {
77
useEffect(() => {
88
if (isDev() && condition()) {
99
// eslint-disable-next-line no-console

stories/Table.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ export const TablePanel = (): JSX.Element => (
5353
</Table.Panel>
5454
);
5555

56-
export const ResponsiveTable = () => (
56+
export const ResponsiveTable = (): JSX.Element => (
5757
<Table responsive caption="Ibuprofen syrup dosages for children">
5858
<Table.Head role="rowgroup">
5959
<Table.Row>

0 commit comments

Comments
 (0)