Skip to content

Comments

Allow and prefer non-prefixed extra fields for AzureFileShareHook#27041

Merged
dstandish merged 2 commits intoapache:mainfrom
astronomer:azure-fileshare-no-prefix
Oct 28, 2022
Merged

Allow and prefer non-prefixed extra fields for AzureFileShareHook#27041
dstandish merged 2 commits intoapache:mainfrom
astronomer:azure-fileshare-no-prefix

Conversation

@dstandish
Copy link
Contributor

From 2.3, non-prefixed extras are fully supported so we make this the preferred way.

Copy link
Contributor

@ferruzzi ferruzzi left a comment

Choose a reason for hiding this comment

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

Left a non-blocking question/suggestion, otherwise LGTM.

Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure what eff_key is here, maybe that's a known term I'm just not familiar with, but it looks like it's the short (non-prefixed) key name, so maybe just call it that?

@dstandish dstandish changed the title Allow non-prefixed extras in AzureFileShareHook Allow and prefer non-prefixed extra fields for AzureFileShareHook Oct 22, 2022
@dstandish dstandish force-pushed the azure-fileshare-no-prefix branch from 05b96fc to 0190c60 Compare October 24, 2022 05:37
From 2.3, non-prefixed extras are fully supported so we make this the preferred way.
@dstandish dstandish force-pushed the azure-fileshare-no-prefix branch from 0190c60 to 9b2cf94 Compare October 24, 2022 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants