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

chore: Refactor and update #161

Merged
merged 6 commits into from
Feb 7, 2025
Merged

chore: Refactor and update #161

merged 6 commits into from
Feb 7, 2025

Conversation

jhnns
Copy link
Member

@jhnns jhnns commented Feb 7, 2025

No description provided.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 13209597242

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

  • 241 of 509 (47.35%) changed or added relevant lines in 29 files are covered.
  • 337 unchanged lines in 20 files lost coverage.
  • Overall coverage decreased (-6.9%) to 75.551%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/exports/csv.exports.ts 0 1 0.0%
src/models/lumpsumService.mocks.ts 2 3 66.67%
src/models/team.mocks.ts 0 1 0.0%
src/lib/mappings.ts 17 19 89.47%
src/models/entry.ts 2 4 50.0%
src/models/holidaysCarryover.mocks.ts 0 2 0.0%
src/models/overtimecarry.mocks.ts 0 2 0.0%
src/models/project.mocks.ts 7 9 77.78%
src/models/service.mocks.ts 0 2 0.0%
src/models/customer.mocks.ts 0 3 0.0%
Files with Coverage Reduction New Missed Lines %
src/lib/requiredParams.ts 1 98.11%
src/models/project.mocks.ts 2 78.05%
src/models/targethours.mocks.ts 2 98.05%
src/lib/mocks.ts 4 93.55%
src/models/workTimes.mocks.ts 5 94.53%
src/models/team.mocks.ts 5 0.0%
src/models/entry.mocks.ts 8 87.25%
src/models/customer.mocks.ts 9 0.0%
src/lib/api.ts 9 91.29%
src/models/service.mocks.ts 10 0.0%
Totals Coverage Status
Change from base Build 13200818765: -6.9%
Covered Lines: 1929
Relevant Lines: 2606

💛 - Coveralls

@jhnns jhnns changed the base branch from main to beta February 7, 2025 23:18
@jhnns jhnns merged commit 859e894 into beta Feb 7, 2025
1 check passed
@jhnns jhnns deleted the update-deps branch February 7, 2025 23:26
github-actions bot pushed a commit that referenced this pull request Feb 7, 2025
# [25.0.0-beta.1](v24.0.0...v25.0.0-beta.1) (2025-02-07)

### chore

* Refactor and update ([#161](#161)) ([859e894](859e894))

### BREAKING CHANGES

* Changed `api` `config` method to a setter and getter property
* Removed Axios options from `api` methods. We plan to remove Axios in the future. You can pass headers instead of Axios options as third parameter to `post`, `put` and `delete` now.
* Updated TypeScript which might affect the generated code.
* Updated Faker and mocks. This will change the produced mock data in case you snapshotted it.
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