Skip to content
This repository has been archived by the owner on Jan 20, 2024. It is now read-only.

Add 'all' as the way to install both chaos and agents subpackage together #88

Merged
merged 2 commits into from
Nov 3, 2021

Conversation

shashankrnr32
Copy link
Member

@shashankrnr32 shashankrnr32 commented Nov 3, 2021

Summary

@yahoo/ychaos-dev

This is a Pull Request that is capable of changing the world.


Fixes: #{issue_number}

Checklist

Checklist (Developer)

Prerequisites

  • I have read the contribution guidelines

Code Analysis

  • Covered by Unittests
  • Security warnings ignored (Why?!)

Project related

  • Schema changed and is backward compatible.
  • Introduces a new sub-command for ychaos CLI

Autogenerated Files

  • Auto generated schema

Pre-Merge Checklist

  • All the build jobs are passing

Checklist (Reviewer 1)

  • Reviewed Source Code
  • Reviewed Tests (If applicable)
  • Reviewed Documentation (If applicable)

Checklist (Reviewer 2)

  • Reviewed Source Code
  • Reviewed Tests (If applicable)
  • Reviewed Documentation (If applicable)

@shashankrnr32 shashankrnr32 requested a review from a team as a code owner November 3, 2021 10:34
@shashankrnr32 shashankrnr32 self-assigned this Nov 3, 2021
@shashankrnr32 shashankrnr32 added the dependencies This issue/pull request is regarding the dependencies label Nov 3, 2021
@codecov
Copy link

codecov bot commented Nov 3, 2021

Codecov Report

Merging #88 (1e3d029) into main (e1c1370) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #88   +/-   ##
=======================================
  Coverage   98.67%   98.67%           
=======================================
  Files          56       56           
  Lines        2424     2424           
  Branches      242      242           
=======================================
  Hits         2392     2392           
  Misses         20       20           
  Partials       12       12           

@shashankrnr32 shashankrnr32 merged commit 67ee262 into yahoo:main Nov 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies This issue/pull request is regarding the dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants