Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use new version of c2c.template #2160

Merged
merged 1 commit into from
Apr 25, 2016
Merged

Use new version of c2c.template #2160

merged 1 commit into from
Apr 25, 2016

Conversation

sbrunner
Copy link
Member

@sbrunner sbrunner commented Apr 14, 2016

@sbrunner sbrunner added this to the 2.1 milestone Apr 14, 2016
@@ -77,13 +77,13 @@ def __call__(self, filename, options):
with open(filename) as config_file:
config = yaml.load(config_file)
# for application config (.build/config.yaml)
if "raster" in config:
if "vars" in config:
Copy link
Contributor

Choose a reason for hiding this comment

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

Not 100% equivalent. Are you sure you don't want that?

if "vars" in config and "raster" in config["vars"]:

Copy link
Member Author

Choose a reason for hiding this comment

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

yes it's not 100% equivalent but the latest test is a little bit a hack, I prefer the new one :-)

@sbrunner sbrunner merged commit ff7b8ee into master Apr 25, 2016
@sbrunner sbrunner deleted the c2ctemplate-environ branch April 25, 2016 13:44
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