Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

RSV Child Resource Rework #441

Merged
merged 18 commits into from
Nov 12, 2021
Merged

RSV Child Resource Rework #441

merged 18 commits into from
Nov 12, 2021

Conversation

senavar
Copy link
Contributor

@senavar senavar commented Nov 9, 2021

Change

Refactored the RSV module to align with the new Child-Resource structure

Type of Change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

RecoveryServices: Vaults

senavar and others added 12 commits November 5, 2021 11:10
There was a line in the gitIgnore file to ignore Backup/*. As a result, new folders labeled backup were not being tracked. This has been fixed and the rest of the child resources have been pushed.
* Child Resource Re-Work for RSV

* Child Resource Re-Work for RSV + Update gitIgnore

There was a line in the gitIgnore file to ignore Backup/*. As a result, new folders labeled backup were not being tracked. This has been fixed and the rest of the child resources have been pushed.

* Update allowed parameters and parameter file

* Update RSV ReaUpdate RSV ReadMe FilesdMe Files
@github-actions
Copy link

github-actions bot commented Nov 11, 2021

Unit Test Results

  1 files  1 suites   21s ⏱️
  7 tests 7 ✔️   0 💤 0
38 runs  7 ✔️ 31 💤 0

Results for commit 3abf1fd.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@eriqua eriqua left a comment

Choose a reason for hiding this comment

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

🆗

@senavar senavar merged commit 949f42b into main Nov 12, 2021
@senavar senavar deleted the users/senavar/rsvRework branch November 12, 2021 17:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Recovery Services Vault to new child-resource structure Feature Request
2 participants