Skip to content

[FIX]: Fix deno import - #474

Open
Arkapravo-Ghosh wants to merge 1 commit into
ryo-ma:masterfrom
Arkapravo-Ghosh:master
Open

Arkapravo-Ghosh wants to merge 1 commit into
ryo-ma:masterfrom
Arkapravo-Ghosh:master

Conversation

@Arkapravo-Ghosh

Copy link
Copy Markdown

This pull request updates the way environment variables are loaded in the api/index.ts file by switching from a local standard library import to a specific versioned remote URL. This change ensures that the project uses a consistent and explicit version of the Deno standard library for environment variable loading.

Dependency management update:

  • Switched the dotenv import from a bare specifier (@std/dotenv/load) to an explicit remote URL (https://deno.land/std@0.224.0/dotenv/load.ts) in api/index.ts to ensure consistent and reliable environment variable loading.

Copilot AI balanced review requested due to automatic review settings October 6, 2026 21:13

Copilot AI left a comment

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

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.

Copilot review overview

🟢 Approval recommended

The only noted issue is a minor import-map consistency nit.

Review effort: Lite
Findings: None

Sallyn0225 added a commit to Sallyn0225/github-profile-trophy that referenced this pull request Oct 27, 2026

This branch has not been deployed

No deployments
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