Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Move the source code to open source repo #5

Merged
merged 2 commits into from
Apr 3, 2024
Merged

Move the source code to open source repo #5

merged 2 commits into from
Apr 3, 2024

Conversation

s-vitaliy
Copy link
Contributor

@s-vitaliy s-vitaliy commented Apr 3, 2024

Scope

  • The source code has been moved to the open source repo
  • Added docstrings to public classes/methods/etc.
  • Updated project structure

NOTES

  1. The source code was moved as is w/o refactoring, but the project layout was updated.
  2. Helm chart and CRDs was not moved
  3. The code coverage will be updated later, see: Improve code coverage in this repo #6

## Scope

- The source code has been moved to the open source repo
- Added docstrings to public classes/methods/etc.
- Updated project structure

**NOTES**
1. The source code was moved as is w/o refactoring, but project layout was updated.
@s-vitaliy s-vitaliy requested a review from a team as a code owner April 3, 2024 14:00
Copy link

github-actions bot commented Apr 3, 2024

Coverage after merging move-code into main will be

48.60%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   ArcaneEnvironment.cs0%100%0%0%10, 5, 8–9
src/Extensions
   V1JobExtensions.cs85.58%75%90.91%90.16%40, 40, 45, 61–62, 62, 62, 62, 62–65, 83, 83
src/Services
   StreamInteractionService.cs0%100%0%0%16–19, 23–27, 31–35, 39–45
src/Services/Maintenance
   HostedStreamingJobMaintenanceService.cs0%0%0%0%16–24, 27–29, 29, 29–36, 39–42
   StreamingJobMaintenanceService.cs89.84%82.14%100%91.40%103–106, 121, 130–133, 135, 61, 66, 97
src/Services/Operator
   HostedStreamOperatorService.cs82.14%100%66.67%82.61%39–42
   StreamOperatorService.cs93.47%87.04%100%95.65%105–111, 116, 124, 138, 138, 144, 166
src/Services/Repositories
   StreamDefinitionRepository.cs0%0%0%0%100, 100, 100, 100, 100, 100, 100, 100, 100–103, 106–113, 23–30, 33–35, 35, 35, 35, 35, 35, 35, 35, 35–38, 41–50, 54–62, 62, 62, 62, 62, 62, 62, 62, 62–65, 68–77, 80–82, 82, 82, 82, 82, 82, 82, 82, 82–85, 88–95, 98–99
   StreamingJobTemplateRepository.cs0%0%0%0%18–25, 29–31, 31, 31, 31, 31, 31, 31, 31, 31–36, 39–47
src/Services/Streams
   StreamingJobOperatorService.cs0%0%0%0%100–107, 110–113, 113, 113–115, 118–121, 121, 121–123, 126–129, 129, 129–131, 134–137, 140–146, 150–152, 152, 152, 152, 152–155, 155, 155, 155, 155–156, 158–161, 163–169, 173, 173, 173, 173, 173–179, 182–188, 191–200, 31–43, 46, 49–56, 59–60, 60, 60–63, 63, 63–70, 70, 70–90, 90, 90, 90, 90–93, 93, 93–99
src/StreamDefinitions/Base
   IStreamDefinition.cs90%83.33%100%100%28
src/StreamStatuses/StreamStatus/V1Beta1
   V1Beta1StreamStatus.cs100%100%100%100%

@s-vitaliy s-vitaliy merged commit 293942b into main Apr 3, 2024
1 check passed
@s-vitaliy s-vitaliy deleted the move-code branch April 3, 2024 14:42
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