Skip to content

Resource.get_offest_within_root#111

Merged
EdwardLarson merged 1 commit into
redballoonsecurity:masterfrom
whyitfor:maintenance/test_coverage
Nov 17, 2022
Merged

Resource.get_offest_within_root#111
EdwardLarson merged 1 commit into
redballoonsecurity:masterfrom
whyitfor:maintenance/test_coverage

Conversation

@whyitfor
Copy link
Copy Markdown
Contributor

@whyitfor whyitfor commented Nov 17, 2022

Please describe the changes in your request.
Remove Resource.get_offset_within_root

- Remove Resource.get_offest_within_root, an unused method which was originally removed in redballoonsecurity#69
- Mark ResourceViewInterface methods as no cover. They are really abstract methods, but are not marked a such because ResourceViewInterface already has a metaclass
@whyitfor whyitfor force-pushed the maintenance/test_coverage branch from c1a067d to 2b33651 Compare November 17, 2022 17:35
@EdwardLarson EdwardLarson merged commit 7e67df7 into redballoonsecurity:master Nov 17, 2022
whyitfor added a commit that referenced this pull request Dec 15, 2022
- Bump fun-coverage to 0.2.0
- Fix signature of function in resource_model.py
- Mark all methods of ResourceViewInterface as no cover (one was missed in #111)
whyitfor added a commit that referenced this pull request Dec 15, 2022
- Bump fun-coverage to 0.2.0
- Fix signature of function in resource_model.py
- Mark all methods of ResourceViewInterface as no cover (one was missed in #111)
whyitfor added a commit that referenced this pull request Dec 15, 2022
- Bump fun-coverage to 0.2.0
- Fix signature of function in resource_model.py
- Mark all methods of ResourceViewInterface as no cover (one was missed in #111)
- Bump function coverage for ofrak to 97
EdwardLarson pushed a commit that referenced this pull request Dec 16, 2022
- Bump fun-coverage to 0.2.0
- Fix signature of function in resource_model.py
- Mark all methods of ResourceViewInterface as no cover (one was missed in #111)
- Bump function coverage for ofrak to 97
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