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

Minor code cleanup #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Minor code cleanup #127

wants to merge 1 commit into from

Conversation

fykidwai
Copy link

@fykidwai fykidwai commented Jul 5, 2021

  • Merged nested if blocks
  • removed redundant else
  • Removed unnecessary tostring() calls
  • removed unnecessary array creation
  • Converted map.keySet() loops to map.entrySet()
  • Updated .gitignore

@fykidwai
Minor code cleanup
31cf44b 
Merged nested if blocks
removed redundant else
Removed unnecessary tostring() calls
removed unnecessary array creation
Converted map.keySet() loops to map.entrySet()
Updated .gitignore
@fykidwai
Copy link
Author

fykidwai commented Jul 8, 2021

@ala-ebi / @sajo-ebi : Can you please review the PR. it has only minor changes that shouldn't impact any functionality

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fykidwai