Skip to content

Feature: Add pre-push check to stop pushing outside time limits#45

Open
fapdash wants to merge 5 commits intoEMPRI-DEVOPS:masterfrom
fapdash:add-limit-push-check
Open

Feature: Add pre-push check to stop pushing outside time limits#45
fapdash wants to merge 5 commits intoEMPRI-DEVOPS:masterfrom
fapdash:add-limit-push-check

Conversation

@fapdash
Copy link
Copy Markdown
Collaborator

@fapdash fapdash commented Feb 20, 2026

  • as pushing will most likely create timestamps outside of the time
    limits on the git forge we want to limit pushing as well
  • allow disabling of the pre-push checks individually via environment
    variables GITPRIVACY_DISABLE_PREPUSH_UNREDACTED and
    GITPRIVACY_DISABLE_PREPUSH_LIMITS

based on #43

TODOs

  • Add test for new prepush limit check

@fapdash fapdash force-pushed the add-limit-push-check branch 3 times, most recently from 20fe762 to bf610f2 Compare February 23, 2026 16:28
@fapdash
Copy link
Copy Markdown
Collaborator Author

fapdash commented Feb 24, 2026

It might be more user friendly to be able to disable the prepush checks via git config? The push might be executed from an editor or IDE where it's not straight forward to quickly set and unset an environment variable for the git push command.

fapdash added 5 commits March 8, 2026 22:27
- as pushing will most likely create timestamps outside of the time
  limits on the git forge we want to limit pushing as well
- allow disabling of the pre-push checks individually via environment
  variables GITPRIVACY_DISABLE_PREPUSH_UNREDACTED and
  GITPRIVACY_DISABLE_PREPUSH_LIMITS

Fix type hint
@fapdash fapdash force-pushed the add-limit-push-check branch from bf610f2 to 1b04221 Compare March 8, 2026 21:28
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.

1 participant