Skip to content

Fix deno import links - #53

Closed
ibanks42 wants to merge 2 commits into
appwrite:mainfrom
ibanks42:fix-52-deno-runtime-appwrite-link
Closed

Fix deno import links#53
ibanks42 wants to merge 2 commits into
appwrite:mainfrom
ibanks42:fix-52-deno-runtime-appwrite-link

Conversation

@ibanks42

Copy link
Copy Markdown

What does this PR do?

Fix deno runtime import links.

Test Plan

  • Run appwrite cli command appwrite init function and choose deno runtime.
  • Go into function/src/deps.ts and change https://deno.land/x/appwrite@6.0.0/mod.ts to https://deno.land/x/appwrite/mod.ts.
  • Deploy function
  • Build succeeds now

Related PRs and Issues

Fix for #52

Have you read the Contributing Guidelines on issues?

Yes.

@loks0n

Copy link
Copy Markdown
Member

I think it'd be better to stay pinned to a specific version e.g. appwrite@7.0.0

@ibanks42

Copy link
Copy Markdown
Author

I think it'd be better to stay pinned to a specific version e.g. appwrite@7.0.0

Fair enough, changed to appwrite@7.0.0

@loks0n
loks0n changed the base branch from main to devOctober 24, 2023 07:46
@loks0n
loks0n changed the base branch from dev to mainOctober 24, 2023 07:47
@loks0nloks0n closed this Jun 10, 2024
Sign up for freeto 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

@ibanks42@loks0n