Skip to content

Jira prod support#2

Open
wor-thongthai wants to merge 14 commits intomainfrom
jira-prod-support
Open

Jira prod support#2
wor-thongthai wants to merge 14 commits intomainfrom
jira-prod-support

Conversation

@wor-thongthai
Copy link

No description provided.

@wor-thongthai wor-thongthai requested a review from Copilot April 7, 2025 19:18
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

examples/jira_api_integration.py:196

  • Importing 'time' inside the function is redundant since imports are usually placed at the top; consider moving this import for consistency.
import time

Comment on lines +25 to +26
# Load environment variables, first trying .env.local
load_dotenv(dotenv_path='.env.local') # Try local env first
Copy link

Copilot AI Apr 7, 2025

Choose a reason for hiding this comment

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

The environment variables are already loaded on line 17; consider removing this duplicate call to avoid potential conflicts.

Suggested change
# Load environment variables, first trying .env.local
load_dotenv(dotenv_path='.env.local') # Try local env first
# Load environment variables if not already loaded

Copilot uses AI. Check for mistakes.
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