GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Timezone find and fix and sonar codemod by clavedeluna · Pull Request #802 · pixee/codemodder-python

Conversation

@clavedeluna

@clavedeluna clavedeluna commented Aug 19, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Overview

New fix and fix and sonar codemod to replace two deprecated datetime functions for ones that are timezone aware

Closes #589
Related to #385, but that one would add to this codemod

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@clavedeluna clavedeluna mentioned this pull request Aug 19, 2024
Codemod to add tz=UTC to any datetime function without a timezone #385
Open
@pixeebot pixeebot Bot mentioned this pull request Aug 19, 2024
Hardening suggestions for codemodder-python / sonar-timezone #803
Merged
@clavedeluna
clavedeluna force-pushed the sonar-timezone branch from 409d4ad to 6de9bf6 Compare August 19, 2024 14:02
@clavedeluna
clavedeluna marked this pull request as ready for review August 19, 2024 14:03
@clavedeluna
clavedeluna requested review from andrecsilva and drdavella as code owners August 19, 2024 14:03
@pixeebot pixeebot Bot mentioned this pull request Aug 19, 2024
Hardening suggestions for codemodder-python / sonar-timezone #804
Merged
andrecsilva
andrecsilva approved these changes Aug 20, 2024
View reviewed changes
drdavella
drdavella approved these changes Aug 29, 2024
View reviewed changes
@clavedeluna
clavedeluna added this pull request to the merge queue Aug 30, 2024
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Aug 30, 2024
clavedeluna added 3 commits August 30, 2024 14:26
@clavedeluna
add core codemod
f6c0ccb
@clavedeluna
document
1a86c75
@clavedeluna
add sonar codemod
ccc7f75
@clavedeluna
clavedeluna force-pushed the sonar-timezone branch from c6d1b04 to 07c7c89 Compare August 30, 2024 17:27
@clavedeluna
clavedeluna enabled auto-merge August 30, 2024 17:27
@pixeebot pixeebot Bot mentioned this pull request Aug 30, 2024
Hardening suggestions for codemodder-python / sonar-timezone #818
Merged
@clavedeluna
update pygoat findings
88cf784
@clavedeluna
clavedeluna force-pushed the sonar-timezone branch from 07c7c89 to 88cf784 Compare August 30, 2024 17:31
@pixeebot @clavedeluna
Hardening suggestions for codemodder-python / sonar-timezone (#818)
e6da90f 
* update pygoat findings

* Use Assignment Expression (Walrus) In Conditional

---------

Co-authored-by: clavedeluna <[email protected]>
Co-authored-by: pixeebot[bot] <104101892+pixeebot[bot]@users.noreply.github.com>
@sonarqubecloud

sonarqubecloud Bot commented Aug 30, 2024

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud
pixeebot[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@clavedeluna
clavedeluna added this pull request to the merge queue Aug 30, 2024
Merged via the queue into main with commit ad2828c Aug 30, 2024
@clavedeluna
clavedeluna deleted the sonar-timezone branch August 30, 2024 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@drdavella drdavella drdavella approved these changes
@andrecsilva andrecsilva andrecsilva approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Codemod for deprecated datetime.utcnow & datetime.utcfromtimestamp

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@clavedeluna @drdavella @andrecsilva
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.