Skip to content

Fix/app token - #304

Merged
a5chin merged 2 commits into
developfrom
fix/app-token
Sep 18, 2026
Merged

a5chin merged 2 commits into
developfrom
fix/app-token

Conversation

@a5chin

@a5chin a5chin commented Sep 17, 2026

Copy link
Copy Markdown
Owner

User description

Type of Change

  • Hotfix
  • Bug Fix
  • Dependency Update
  • Feature
  • Refactor
  • CI/CD
  • Documentation

Related Issues

Breaking Changes

  • No breaking changes
  • API signature changed
  • Configuration changed
  • Behavior changed
  • Dependencies changed
  • Features removed

Checklist

  • Dependencies added/updated
  • New environment variables
  • Performance impact assessed
  • Security implications reviewed
  • Deployment notes

Additional Context


PR Type

Bug fix, Enhancement


Description

  • Update GitHub App token parameter app-id to client-id.

  • Upgrade Codecov action to version 7.

  • Configure report_type for Codecov test results.


File Walkthrough

Relevant files
Configuration changes
setting.yml
Update GitHub App token parameter                                               

.github/workflows/setting.yml

  • Replaced the app-id input with client-id for the
    actions/create-github-app-token@v3 action.
  • Applied this parameter update consistently across all jobs (branch,
    release, pages, permission, protection) that generate a GitHub App
    token.
+5/-5     
Ci/cd
test.yml
Update GitHub App token and Codecov action                             

.github/workflows/test.yml

  • Replaced the app-id input with client-id for the
    actions/create-github-app-app-token@v3 action.
  • Upgraded the Codecov action from codecov/test-results-action@v1 to
    codecov/codecov-action@v7.
  • Added the report_type: test_results configuration to the updated
    Codecov action.
+3/-2     

@a5chin a5chin self-assigned this Sep 17, 2026
@github-actions github-actions Bot added the fix label Sep 17, 2026
@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (738ca19) to head (c4bca6c).
⚠️ Report is 3 commits behind head on develop.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           develop      #304   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            9         9           
  Lines          103       103           
  Branches         1         1           
=========================================
  Hits           103       103           
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 738ca19...c4bca6c. Read the comment docs.

@a5chin
a5chin merged commit 3ea74d6 into develop Sep 18, 2026
18 checks passed
@a5chin
a5chin deleted the fix/app-token branch September 18, 2026 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant