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

[Config] Use one global config #63

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

benedikt-voelkel
Copy link
Collaborator

A global config key 'config' can be used to pass a dictionary which will be available everywhere.

Per step, it can be extended or changed by passing a dict in that stage, also using the key 'config'

@benedikt-voelkel
Copy link
Collaborator Author

This simplifies the usage of the global configuration. Before, it would have been required to copy the full config into each individual config of a stage.

@benedikt-voelkel
Copy link
Collaborator Author

d = d1 | d2

for dictionaries only works from python 3.9 onwards... Maybe 3.8 is simply too old to be worth testing anymore?!

@mconcas
Copy link
Collaborator

mconcas commented Oct 5, 2024

Yep, python 3.8 is eol since 4y+. I'd remove it from matrix test.

A global config key 'config' can be used to pass a dictionary
which will be available everywhere.

Per step, it can be extended or changed by passing a dict in that stage,
also using the key 'config'
@benedikt-voelkel
Copy link
Collaborator Author

Ok, this one works now as well.
Next one will be an update for the README

@mconcas mconcas merged commit aa5c5a8 into AliceO2Group:master Oct 9, 2024
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