Skip to content

fix(engines): add nft annotation for libquery_engine#6

Merged
Jolg42 merged 4 commits intomasterfrom
nft/libquery_engine
Mar 2, 2021
Merged

fix(engines): add nft annotation for libquery_engine#6
Jolg42 merged 4 commits intomasterfrom
nft/libquery_engine

Conversation

@williamluke4
Copy link
Copy Markdown

No description provided.

@williamluke4 williamluke4 force-pushed the nft/libquery_engine branch from 0d7b6e4 to b2c9e6c Compare March 1, 2021 14:00
Comment thread packages/engines/src/index.ts Outdated
import { download } from '@prisma/fetch-engine'
import { enginesVersion } from "@prisma/engines-version";
import { download } from "@prisma/fetch-engine";
import path from "path";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why the double quotes and ; made it here.

Would be good to have the same setting as here https://github.com/prisma/prisma/blob/master/src/.prettierrc.yml

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah prettier just needs to be setup. I Will add that in this PR

@Jolg42
Copy link
Copy Markdown
Contributor

Jolg42 commented Mar 1, 2021

Should we merge this today?

@williamluke4
Copy link
Copy Markdown
Author

williamluke4 commented Mar 1, 2021 via email

Copy link
Copy Markdown
Contributor

@Jolg42 Jolg42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding Prettier ❤️

@Jolg42 Jolg42 merged commit 13b488d into master Mar 2, 2021
@Jolg42 Jolg42 deleted the nft/libquery_engine branch March 2, 2021 15:59
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.

2 participants