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

Apply final improvements #69

Merged
merged 4 commits into from
Apr 17, 2023
Merged

Apply final improvements #69

merged 4 commits into from
Apr 17, 2023

Conversation

L-Zuluaga
Copy link
Member

@L-Zuluaga L-Zuluaga commented Apr 17, 2023

Description

Apply minor refactor to initialize and distribute functions, and update README file with the suggested improvements.

Type of change

  • This change requires a documentation update

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented on my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my change

@L-Zuluaga L-Zuluaga self-assigned this Apr 17, 2023
@L-Zuluaga L-Zuluaga requested a review from einerzg April 17, 2023 15:48
@L-Zuluaga L-Zuluaga marked this pull request as ready for review April 17, 2023 15:51
@L-Zuluaga L-Zuluaga merged commit 2fbba26 into v0.4 Apr 17, 2023
1 check passed
@L-Zuluaga L-Zuluaga deleted the apply-final-improvements branch April 17, 2023 21:20
@L-Zuluaga L-Zuluaga linked an issue Apr 18, 2023 that may be closed by this pull request
@coveralls
Copy link

Pull Request Test Coverage Report for Build 4723821596

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 16 of 16 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 4704385739: 0.0%
Covered Lines: 293
Relevant Lines: 293

💛 - Coveralls

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.

Functions refactor and readme improves
3 participants