Skip to content

Makefile tests for else ifeq|ifneq|... syntax#170888

Merged
alexr00 merged 4 commits intomicrosoft:mainfrom
fadeevab:main
Jan 12, 2023
Merged

Makefile tests for else ifeq|ifneq|... syntax#170888
alexr00 merged 4 commits intomicrosoft:mainfrom
fadeevab:main

Conversation

@fadeevab
Copy link
Contributor

@fadeevab fadeevab commented Jan 9, 2023

@alexr00 @jrieken Hi folks, this is to test fadeevab/make.tmbundle@1d4c0b5

Merge this PR after pulling the fresh Makefile syntax.

Before:
image

After:
image

WHY: Found an original complaint about else ifeq

@alexr00 alexr00 assigned alexr00 and unassigned aeschli Jan 10, 2023
@fadeevab
Copy link
Contributor Author

fadeevab commented Jan 11, 2023

@alexr00 Would you be able to add an updated makefile test result file into this PR? I have problems to run tests on the WSL2 locally, I'm afraid it will take too much time for me.

Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the test!

@alexr00 alexr00 added this to the January 2023 milestone Jan 12, 2023
@alexr00 alexr00 merged commit 82305d3 into microsoft:main Jan 12, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Feb 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants