Skip to content

Fix task id generation for tutorials - #272

Open
tnagorra wants to merge 1 commit into
developfrom
fix/tutorial-task-id
Open

Fix task id generation for tutorials#272
tnagorra wants to merge 1 commit into
developfrom
fix/tutorial-task-id

Conversation

@tnagorra

@tnagorra tnagorra commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Depends on

Changes

  • use zero index to calculate task id
  • add unit test for the calculation

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • n+1 queries
  • flake8 issues
  • print
  • typos
  • unwanted comments

This PR contains valid:

  • tests

- add unit test for the calculation
@tnagorra
tnagorra marked this pull request as ready for review August 3, 2026 04:52
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.50%. Comparing base (8c053fb) to head (d8a6a5c).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #272      +/-   ##
===========================================
+ Coverage    87.45%   87.50%   +0.05%     
===========================================
  Files          211      213       +2     
  Lines        12837    12877      +40     
  Branches      1089     1085       -4     
===========================================
+ Hits         11226    11268      +42     
+ Misses        1212     1211       -1     
+ Partials       399      398       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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.

1 participant