-
Notifications
You must be signed in to change notification settings - Fork 15
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
Improve Pulumi error messages #1624
Improve Pulumi error messages #1624
Conversation
It looks like this has been based off an older branch of |
It shouldn't be - I updated my local copy of python-migration and merged it into this branch just before making the PR Edit: ah, which makes it a mess and hard to see what I actually changed, right |
I've merged |
e5f7642
to
283f0e9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
39dc0d2
into
alan-turing-institute:python-migration
✅ Checklist
Enable foobar integration
rather than515 foobar
).develop
.'[WIP]'
to the title if needed (if you're not yet ready to merge)../tests/AutoFormat_Powershell.ps1 -TargetPath <path to file or directory>
for Powershell).Fixes incorrect error messages for the deployment
name
argument passed todsh init
Adds an error when the backend has not been initialised before attempting to run commands that rely on it (e.g.
dsh admin list-users
)Note - this PR also updates the relevant .devcontainer to Pulumi version
3.80.0
🌂 Related issues
Closes #1623
🔬 Tests
Tested locally: