Support Negative Values for days_after Parameter in Contribution Script #35
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.
Overview
This introduces the ability to use negative values for the
days_after
parameter in the GitHub contribution generation script. This enhancement increases the flexibility of the script, allowing users to specify a commit history that ends before the current date.Changes
days_after
to ensure the sum ofdays_before
anddays_after
is non-negative.days_after
, enabling the script to create a commit history that can end before the current date.Impact
This change allows users to have more control over the date range for commit generation. It is particularly useful for scenarios where the user wants to backdate contributions but stop at a certain point before the current date.
Examples
Example 1: Standard Usage
days_before = 30
days_after = 10
This would generate commits from 30 days before the current date to 10 days after the current date.
Example 2: Using Negative
days_after
days_before = 30
days_after = -5
This would generate commits from 30 days before the current date and stop 5 days before the current date. It's particularly useful for creating a commit history that doesn't include the most recent days.
Credit to @jeremiah-shore who gave me this idea from a suggestion