Skip to content

Support highly nested project structure when generating build tasks#61

Merged
nblagoev merged 1 commit intomasterfrom
bugfix/support-highly-nested-projects
Feb 9, 2020
Merged

Support highly nested project structure when generating build tasks#61
nblagoev merged 1 commit intomasterfrom
bugfix/support-highly-nested-projects

Conversation

@nblagoev
Copy link
Copy Markdown
Contributor

@nblagoev nblagoev commented Feb 6, 2020

Description

Fixed a bug where it is not possible to build a list of vRealize tasks (Cmd+Shift+B) for composite project of multiple nested maven modules.

Checklist

  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have tested against live vRO/vRA, if applicable
  • My changes have been rebased and squashed to the minimal number of relevant commits
  • My changes have a descriptive commit message with a short title, including a Fixed #XXX - or Closed #XXX - prefix to auto-close the issue

Testing

Tested using a project with the following structure.

  • someprj (pom.xml - base-package)
    • common (pom.xml - typescript-project)
    • cm (pom.xml - base-package)
      • vra (pom.xml - vra-ng-package)
      • vro (pom.xml - typescript-project)
    • pod (pom.xml - base-package)
      • vra (pom.xml - vra-ng-package)
      • vro (pom.xml - typescript-project)

Release Notes

Support highly nested project structure when generating build tasks

Signed-off-by: Nikolay Blagoev <blagoevn@vmware.com>
@nblagoev nblagoev requested a review from a team as a code owner February 6, 2020 15:37
@nblagoev nblagoev merged commit 7901ef1 into master Feb 9, 2020
@nblagoev nblagoev deleted the bugfix/support-highly-nested-projects branch February 9, 2020 13:14
VenelinBakalov added a commit that referenced this pull request Feb 11, 2026
[dependencies] (security alert #60, #61, #64) Fix `form-data`, `js-yaml` and `tmp` vulnerabilities
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