Skip to content

Conversation

millin
Copy link
Contributor

@millin millin commented Jul 29, 2025

No description provided.

Comment on lines -100 to -117
[tool.black]
line-length = 88
target-version = ["py39", "py310", "py311", "py312"]
include = '\.pyi?$'
exclude = '''
/(
\.eggs
| \.git
| \.hg
| \.mypy_cache
| \.tox
| \.venv
| _build
| buck-out
| build
| dist
)/
'''
Copy link
Owner

Choose a reason for hiding this comment

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

Thanks for removing this

Comment on lines +31 to +46
def try_decode_base64(s: str) -> str:
"""Attempt to decode a string from base64.
If the string is not valid base64, returns the original value.
Args:
s: The string to decode.
Returns:
The decoded string, or the original value if decoding fails.
"""
try:
s = base64.b64decode(s, validate=True).decode("utf-8")
except binascii.Error:
pass
return s
Copy link
Owner

Choose a reason for hiding this comment

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

Nice!

Copy link
Owner

@tomasfarias tomasfarias left a comment

Choose a reason for hiding this comment

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

I've bumped uv.lock both to review it and to get the new airflow-dbt-python in there.

This will be good to merge once CI is green. Thank you for your contribution.

@tomasfarias tomasfarias merged commit 395ddd9 into tomasfarias:master Jul 29, 2025
27 checks passed
@tomasfarias
Copy link
Owner

I will cut a release for 3.0.3 later today.

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