Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Voicu aws/refresh datasets command #754

Merged
merged 14 commits into from
Mar 14, 2024

enable colors for windows

10ea318
Select commit
Loading
Failed to load commit list.
Merged

Voicu aws/refresh datasets command #754

enable colors for windows
10ea318
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeGuru Reviewer Scanner succeeded Mar 14, 2024 in 3s

1 new alert including 1 low severity security vulnerability

New alerts in code changed by this pull request

Security Alerts:

  • 1 low

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 973 in cid/helpers/quicksight/__init__.py

See this annotation in the file changed.

Code scanning / CodeGuru Reviewer Scanner

Time zone aware datetimes Low

The naive datetime objects are treated by many datetime methods as local times, it is preferred to use aware datetimes to represent times in UTC. The recommended way to create an aware datetime object representing a specific timestamp in UTC is by passing tzinfo as an argument to the method.

Learn more

Check failure on line 74 in cid/cli.py

See this annotation in the file changed.

Code scanning / CodeGuru Reviewer Scanner

Outdated subprocess module API High

This line of code makes an outdated API call to start and communicate with processes. We recommend that you use the subprocess module to start new processes, connect with their pipes, and get their return codes.

Learn more