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

Fix/360 run action build current wrong path #399

Merged
merged 9 commits into from
Mar 19, 2024

Conversation

edmundreinhardt
Copy link
Member

fixes issue 360

When the runAction command is run from the project heading in the Project Explorer view, (see below)
image
the path passed in to the Code 4 i runAction command is the path of the file in the active editor (if that file is in the project), otherwise it is the path of the project.

By this logic a Build All command will succeed not matter what file is in the active editor but Build current will not work in that scenario as it is not clear what is meant.

Thanks Liam for writing the code

Signed-off-by: Edmund Reinhardt <edmund.reinhardt@ca.ibm.com>
Signed-off-by: Edmund Reinhardt <edmund.reinhardt@ca.ibm.com>
Signed-off-by: Edmund Reinhardt <edmund.reinhardt@ca.ibm.com>
… - with tests

Signed-off-by: Edmund Reinhardt <edmund.reinhardt@ca.ibm.com>
Signed-off-by: Edmund Reinhardt <edmund.reinhardt@ca.ibm.com>
Needs to survive being persisted in .env files
Needs to be validate as valid object
Needs to be escaped before being pased to `liblist` PASE command

Signed-off-by: Edmund Reinhardt <edmund.reinhardt@ca.ibm.com>
Signed-off-by: Edmund Reinhardt <edmund.reinhardt@ca.ibm.com>
Signed-off-by: Edmund Reinhardt <edmund.reinhardt@ca.ibm.com>
@edmundreinhardt edmundreinhardt linked an issue Mar 13, 2024 that may be closed by this pull request
Base automatically changed from fix/377-support-doublequoted-libraries to main March 19, 2024 13:18
Because this pull request was requested before the double quoted name
fix was meged into main, but the fixes were present in this branch, so
git thinks it is a conflict
@edmundreinhardt edmundreinhardt merged commit bb748d5 into main Mar 19, 2024
4 checks passed
@edmundreinhardt edmundreinhardt deleted the fix/360-runAction-buildCurrent-wrongPath branch March 19, 2024 13:26
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.

Build Current Action for ibmi-bob failing
1 participant