Skip to content

Conversation

@acro5piano
Copy link

Hi, thank you for the great library.

Using TypeScript 4.3.2, there are errors on this library because it includes .ts file. It should not be published.
This behavior cannot be changed even if tsconfig.json set to the following:

{
  "compilerOptions": {
    "skipLibCheck": true, /* Skip type checking of declaration files. */
  },
  "exclude": [
    "node_modules"
  ]

image

Best regards

@coveralls
Copy link

Pull Request Test Coverage Report for Build 72

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 70: 0.0%
Covered Lines: 124
Relevant Lines: 124

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 72

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 70: 0.0%
Covered Lines: 124
Relevant Lines: 124

💛 - Coveralls

@abrkn
Copy link

abrkn commented Oct 7, 2022

@acro5piano Did you find any workaround for this? I'm having the same issue

@acro5piano
Copy link
Author

@abrkn Deleting the file after install seems to be the only workaround for now.

https://github.com/acro5piano/knex-graphql-utils/blob/master/package.json#L20

@Mikhus
Copy link
Owner

Mikhus commented Nov 9, 2024

I'm running this library in many production projects without any problem described here for years. I don't know how actual it is, so I'm closing it for now. If it is still actual - feel free to re-open this issue, I promise I'll take a deeper look into it.

@Mikhus Mikhus closed this Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants