-
Notifications
You must be signed in to change notification settings - Fork 39
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
texei:package:dependencies:install command requires DevHub #57
Comments
Thanks I'll have a look, but I don't think it's needed. Could be one of these 2 reasons:
I'll make sure it's not needed and removed it, thanks for the feedback ! |
@ImJohnMDaniel looks like the DevHub is used here. I think it's in the case where you define your dependency with a Package Id and a Version Number, instead of a Package Version Id. It then need to query the DevHub to retrieve the corresponding 04t Id. So if it's you use this:
instead of this:
I guess we may mostly be using the 2nd notation now as it's easier with aliases, so it could be something from pre-aliases time. What could be done though is to remove the requirement, and display an error message only in the case of a dependency found with Do you think it's worth it ? |
Hey @FabienTaillon -- apologies for not circling back with you here, but that is exactly what I remembered. In the end, I did not make the parameter a requirement because it is only relevant if you have a non-pinned dependency setup. I would recommend making the parameter optional with an error message should it be discovered that one of the dependencies in the |
Makes sense, I'll flag this as an enhancement. Thanks for the suggestion. |
Hello @FabienTaillon My problem is that I don't want to specify a dev-hub, I just want package installation and nothing more. Any tips to get rid of this ? (moreover our Unlock packages have a lot of Different Dev Hubs so define "One" is not an option for us) PS: We are using package version id (04t) |
This is weird because Dev Hub was already required before. What changed since a few days is our migration to sf V2, that moves --targetdevhubusername to --target-dev-hub flag, but both should still work. Do you have the whole command you're running please ? |
Any chance the parameter requiresDevhubUsername can be set on CLI params ? (I imagine that the answer is No as there is no mention in the doc)
And here the Auth part for the Org:
PS: I moved to sf-v2 (previously we were on sfdx only) the whole thing after getting the same error on the old one So i was getting same error with this one : |
No you can't. I tried and I couldn't reproduce it, I mean, I'm getting the same behavior before the V2 migration and after. I could update the code to require it only in a specific scenario, but that will take some time to update. Are you sure you did not have any DevHub de setup previously ? |
To be more precise the first error I got (2 days ago Tuesday 26th of September) from the texei CLI is "Error (1): The specified org CI is not a Dev Hub". Could you (as a quick workaround) keep the code "as is" (requiresDevhubUsername default to True) and let us set it to False through a new parameter of the CLI? or has an environment variable like for SF CLI e.g :
Then as a long term solution, make it required only for specific scenario. |
@arafesthain I've removed the Dev Hub as mandatory, and only checking for it when needed, as @ImJohnMDaniel suggested, can you give it a try ? |
@FabienTaillon thanks a lot! working perfectly 😃 |
Perfect ! Closing this one then. |
I noticed that the
texei:package:dependencies:install
command requires the DevHub to be specified. Is that really required?@FabienTaillon, in my version of the command, it was simply because line 41 was set to true. Once I changed that to false, the command worked fine. I am not aware of a need in the Texei version to have access to the DevHub so I wanted to check with you first.
The use case where this becomes an issue is with customer admin teams trying to just deploy packages in bulk but they don't use a DevHub themselves.
Let me know if you have questions.
Cheers!
The text was updated successfully, but these errors were encountered: