stolostron / ansible-collection.core

OCM/ACM Ansible Collection
Apache License 2.0
19 stars 13 forks source link

Rename stolostron core #139

Closed nathanweatherly closed 2 years ago

nathanweatherly commented 2 years ago

renaming collection from "ocmplus.cm" to "stolostron.core"

renaming GH repo from "ocmplus.cm" to "ansible-collection.core"

codecov-commenter commented 2 years ago

Codecov Report

Merging #139 (3ff074d) into main (463f87b) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #139   +/-   ##
=======================================
  Coverage   51.54%   51.54%           
=======================================
  Files          19       19           
  Lines        1490     1490           
  Branches      201      201           
=======================================
  Hits          768      768           
  Misses        717      717           
  Partials        5        5           
Flag Coverage Δ
sanity 30.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...odule_utils/managedcluster_addons/cluster_proxy.py
.../unit/plugins/module_utils/test_installer_utils.py
...ocmplus/cm/plugins/inventory/ocm_managedcluster.py
...lugins/modules/test_managed_serviceaccount_rbac.py
...s/module_utils/managedcluster_addons/addon_base.py
...le_utils/managedcluster_addons/search_collector.py
...ctions/ocmplus/cm/plugins/modules/cluster_proxy.py
...e_utils/managedcluster_addons/policy_controller.py
...mplus/cm/plugins/modules/managed_serviceaccount.py
...ls/managedcluster_addons/cert_policy_controller.py
... and 28 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 463f87b...3ff074d. Read the comment docs.

openshift-ci[bot] commented 2 years ago

New changes are detected. LGTM label has been removed.

openshift-ci[bot] commented 2 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nathanweatherly, TheRealHaoLiu

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/stolostron/ansible-collection.core/blob/main/OWNERS)~~ [TheRealHaoLiu] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
TheRealHaoLiu commented 2 years ago

merging PR without integration test due to big environmental change that will cause integration test to not able to run properly from fork. running integration test post merge