Skip to content

Update run-hop-in-apache-airflow.adoc#3024

Closed
dmainou wants to merge 1 commit intoapache:masterfrom
dmainou:patch-1
Closed

Update run-hop-in-apache-airflow.adoc#3024
dmainou wants to merge 1 commit intoapache:masterfrom
dmainou:patch-1

Conversation

@dmainou
Copy link

@dmainou dmainou commented Jun 22, 2023

on the front end the underscores being changed don't show up for the first 2 instances but do for the 3rd.

Turns out the variable had double underscores on the first 2 instances and not on the 3rd (which displays correctly on the front end)

Please add a meaningful description for your change here


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

on the front end the underscores being changed don't show up for the first 2 instances but do for the 3rd.

Turns out the variable had double underscores on the first 2 instances and not on the 3rd (which displays correctly on the front end)
@bamaer
Copy link
Contributor

bamaer commented Jun 23, 2023

thanks a lot for the PR, Diego. the double underscores are what Airflow expects, so can't be reduced to a single one...
closing the PR, but appreciate the contribution nevertheless!

@bamaer bamaer closed this Jun 23, 2023
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.

2 participants