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

Make jobMinDuration and jobMaxDuration required #278

Merged
merged 3 commits into from
Jan 18, 2023

Conversation

marco-vb
Copy link

@marco-vb marco-vb commented Dec 6, 2022

Closes #263

@marco-vb marco-vb requested a review from Naapperas December 6, 2022 17:29
@Naapperas Naapperas changed the title Fixed all tests Make jobMinDuration and jobMaxDuration required Dec 6, 2022
Copy link
Member

@Naapperas Naapperas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. The only thing needed is to pick better commit names

test/end-to-end/company.js Show resolved Hide resolved
@marco-vb marco-vb force-pushed the fix/make-job-durations-required branch 3 times, most recently from 8da60c4 to 6bc287e Compare December 15, 2022 15:13
@codecov-commenter
Copy link

codecov-commenter commented Dec 15, 2022

Codecov Report

Base: 90.50% // Head: 90.46% // Decreases project coverage by -0.05% ⚠️

Coverage data is based on head (a18f5c1) compared to base (eaecd99).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #278      +/-   ##
===========================================
- Coverage    90.50%   90.46%   -0.05%     
===========================================
  Files           54       54              
  Lines         1401     1395       -6     
  Branches       223      222       -1     
===========================================
- Hits          1268     1262       -6     
  Misses         129      129              
  Partials         4        4              
Impacted Files Coverage Δ
src/models/Offer.js 87.87% <ø> (-0.70%) ⬇️
src/api/middleware/validators/offer.js 97.20% <100.00%> (-0.10%) ⬇️
src/api/middleware/validators/validationReasons.js 84.21% <100.00%> (+0.87%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Naapperas
Naapperas previously approved these changes Dec 15, 2022
Copy link
Member

@BrunoRosendo BrunoRosendo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marco-vb Please update the API documentation with this change. It is stored under docs/ and the syntax resembles markdown, you can read the docs (in the README) to know how to use it.

@marco-vb marco-vb force-pushed the fix/make-job-durations-required branch from afcc52e to a18f5c1 Compare January 10, 2023 15:17
Copy link
Member

@Naapperas Naapperas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@BrunoRosendo BrunoRosendo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@marco-vb marco-vb closed this Jan 17, 2023
@marco-vb marco-vb reopened this Jan 17, 2023
@marco-vb marco-vb merged commit 24692a7 into develop Jan 18, 2023
@marco-vb marco-vb deleted the fix/make-job-durations-required branch January 18, 2023 01:05
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.

Make jobMinDuration and jobMaxDuration mandatory
4 participants