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

Ardalan foroughi patch 1 #102

Conversation

ardalanForoughipour
Copy link
Contributor

Description

Proposed solution

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (e.g., refactoring, documentation, etc.)

Additional context

Checklist

  • I have read the contributing guidelines
  • I have performed a self-review of my own code (added comments, corrected misspellings etc.)
  • My code follows the style guidelines of this project
  • I have made corresponding changes to the documentation (if appropriate)
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes

@ardalanForoughipour
Copy link
Contributor Author

Exception handling of Reflected method both in Reflection and AdvancedReflection has been fixed. Also added android/arch to list of libs_to_ignore.txt

@codecov
Copy link

codecov bot commented Jun 20, 2021

Codecov Report

Merging #102 (5fb59e7) into master (f1122f0) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #102   +/-   ##
=======================================
  Coverage   77.57%   77.57%           
=======================================
  Files          49       49           
  Lines        2453     2453           
  Branches      486      486           
=======================================
  Hits         1903     1903           
  Misses        422      422           
  Partials      128      128           
Impacted Files Coverage Δ
...const_string_encryption/const_string_encryption.py 85.29% <100.00%> (ø)

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 dd4c070...5fb59e7. Read the comment docs.

@ClaudiuGeorgiu ClaudiuGeorgiu merged commit d6e527c into ClaudiuGeorgiu:master Jun 20, 2021
@ClaudiuGeorgiu
Copy link
Owner

Thanks @ardalan-foroughi!

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.

2 participants