8000 Fix exception when assigning xml memory bank entry to "memory" variable by slqa · Pull Request #13689 · hashicorp/vagrant · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix exception when assigning xml memory bank entry to "memory" variable #13689

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

slqa
Copy link
@slqa slqa commented Jun 28, 2025

Fixes after this issue #12180 PR 13760

Due to non case sensitive powershell "memory" variable is evaluated as int due to "Memory" function parameter, and this causes an exception

Due to non case sensitive powershell "memory" is evaluated as int due to "Memory" function parameter, and thus an exception
@slqa slqa requested a review from a team as a code owner June 28, 2025 21:06
Copy link
hashicorp-cla-app bot commented Jun 28, 2025

CLA assistant check
All committers have signed the CLA.

@anshulsharma-hashicorp
Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read 8000 and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@slqa you need to sign CLA

@slqa
Copy link
Author
slqa commented Jun 30, 2025

@anshulsharma-hashicorp I thought I did it, on CLA page I click "Sign in with Github to agree", and then I'm landing on main CLA page, and there is nothing to do

edit. somehow other page appeared. done

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
0