Skip to content

Add constants - #82

Merged
kratzert merged 3 commits into
mainfrom
add-constants
Mar 23, 2026
Merged

Add constants#82
kratzert merged 3 commits into
mainfrom
add-constants

Conversation

@thiagovmdon

Copy link
Copy Markdown
Collaborator

No description provided.

@kratzert kratzert left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Minor typo fix required

Comment thread rivretrieve/constants.py Outdated
DAILY = "daily"
HOURLY = "hourly"
INSTANTANEOUS = "instantaneous"
MONTHLY = "monhtly"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Typo

Suggested change
MONTHLY = "monhtly"
MONTHLY = "monthly"

@thiagovmdon

Copy link
Copy Markdown
Collaborator Author

If I would receive some money for every time I make this exact some mistake... Thanks for spotting Freddy! now it should be good :)

@kratzert kratzert left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM

@kratzert
kratzert merged commit 31abf9b into main Mar 23, 2026
6 checks passed
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