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.

Sign up for GitHub

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

Jump to bottom

#750: Created method implementation for each enum constant #762

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

#750: Created method implementation for each enum constant #762

wants to merge 6 commits into from

Conversation

leonrohne27
Copy link
Contributor

Fixes: #750

Implements:

  • Removed switch and created a method implementation for each enum constant instead
  • Changed the method to abstract

@leonrohne27
devonfw#750: Created method implementation for each enum constant
eb9c1e0 
Removed switch and created a method implementation for each enum constant instead and made it abstract
@leonrohne27 leonrohne27 self-assigned this Nov 14, 2024
@coveralls
Copy link
Collaborator

coveralls commented Nov 15, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 11857432623

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.01%) to 67.272%
Files with Coverage Reduction New Missed Lines %
com/devonfw/tools/ide/context/GitUrlSyntax.java 2 84.0%
Totals Coverage Status
Change from base Build 11853076778: -0.01%
Covered Lines: 6399
Relevant Lines: 9170
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@leonrohne27 leonrohne27

Labels
None yet
Projects
IDEasy board
Status: Team Review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Small cleanup from PR #724
4 participants
@leonrohne27 @coveralls @jan-vcapgemini @KianRolf