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

Enhancement: specify minimum stim version in stim.deploy.yaml #58

Closed
mrballcb opened this issue Aug 13, 2020 · 0 comments · Fixed by #60
Closed

Enhancement: specify minimum stim version in stim.deploy.yaml #58

mrballcb opened this issue Aug 13, 2020 · 0 comments · Fixed by #60
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@mrballcb
Copy link

We have seen cases where a dev was having trouble getting stim to work and it ended up being a very old version of stim that had some breaking behavior associated with it. It would be nice if you could specify a minimum stim version somewhere in the stim.deploy.yaml that would immediately bail with an error that the version of stim you are using is less than required by the stim.deploy.yaml. (IMHO it's important to verbosely state that it's specified in the config file as opposed to some obscure, on-the-net check that stim is doing itself against some public "versions" server.)

In the "deployment" section is where this seems to be the best heirarchical placement, but it would require adding some kind of sub-section where you can set minimum and highest versions this config is expected to safely run:

deployment.stim.minVersion
deployment.stim.maxVersion

There may well be much better location for this than in the deployment section.

If no min/max is specified in the config, the checks should probably not be performed at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants