tatethurston/TwirpScript

`import/first` es-lint error

antonioorct opened this issue · 2 comments

image

// THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY.
// Source: auth.proto

import type { ByteSource, ClientConfiguration } from "twirpscript";
import {
  BinaryReader,
  BinaryWriter,
  JSONrequest,
  PBrequest,
} from "twirpscript";
// This is the minimum version supported by the current runtime.
// If this line fails typechecking, breaking changes have been introduced and this
// file needs to be regenerated by running `yarn twirpscript`.
export { MIN_SUPPORTED_VERSION_0_0_34 } from "twirpscript";

import { User } from "./user.pb";
// This line is underlined with the error:
// Import in body of module; reorder to top       eslint(import/first)

Ignoring *.pb.ts files in the eslint config fixes it.

@antonioorct I think it is recommended not to use eslint on the generated files.

Yep! @arranf is correct. I've updated the Readme with more context: https://github.com/tatethurston/TwirpScript#working-with-other-tools. LMK if you have any questions.