-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Create simd/simcli commands + integration tests #5951
Comments
We should totally give simapp another name. I also know @alexanderbez would appreciate this. Possible options from a quick brainstorm on my side:
Personally I think |
I know it will be a bit annoying, but maybe let's create a single comment per proposal and people can 👍 or 👎 the ones they like |
|
|
|
Should we be changing the |
Yes. |
I'll cross check if everything has been ported from gaia/cli-test and close this. @aaronc @alexanderbez do we have a decision on what we want to rename the simapp to? |
I have no strong preference here. @aaronc thoughts? |
How about |
Seemed like people liked atomicd. But maybe we can come up with something better? |
whats wrong with simd & simcli? It is the simplest & self-explanatory. They are only used for sims and testing so they aren't user-facing so naming shouldn't matter. |
I agree with Marko on this. Naming should always try suggest what the program/library is for. |
I agree, let's stick with simd/simcli for now. We have enough stuff in flux to get out for 0.39. And there are user facing naming decisions we do need to deal with |
Being completed here: #6418 |
Summary
Since gaia was moved into a separate repo, there haven't been cli commands in the SDK for simapp nor integration tests. Adding this to the SDK will help with testing in general, and particular help judge completeness of #5456.
For Admin Use
The text was updated successfully, but these errors were encountered: