Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix documentation skip CI#5351

Merged
kan-bayashi merged 1 commit intoespnet:masterfrom
Fhrozen:pr-ci-fix
Jul 22, 2023
Merged

Fix documentation skip CI#5351
kan-bayashi merged 1 commit intoespnet:masterfrom
Fhrozen:pr-ci-fix

Conversation

@Fhrozen
Copy link
Member

@Fhrozen Fhrozen commented Jul 22, 2023

What?

Fix CI variable

Why?

CI were skipping doc verification due to mislabel.

See also

#5320

@mergify mergify bot added the CI Travis, Circle CI, etc label Jul 22, 2023
@sw005320 sw005320 added this to the v.202307 milestone Jul 22, 2023
@codecov
Copy link

codecov bot commented Jul 22, 2023

Codecov Report

Merging #5351 (3811f4d) into master (79a74bc) will decrease coverage by 3.75%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5351      +/-   ##
==========================================
- Coverage   76.11%   72.37%   -3.75%     
==========================================
  Files         672      672              
  Lines       59859    59859              
==========================================
- Hits        45563    43321    -2242     
- Misses      14296    16538    +2242     
Flag Coverage Δ
test_configuration_espnet2 ∅ <ø> (∅)
test_integration_espnet1 65.93% <ø> (ø)
test_integration_espnet2 ?
test_python_espnet1 20.51% <ø> (ø)
test_python_espnet2 51.35% <ø> (ø)
test_utils 23.17% <ø> (ø)

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

see 66 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@kan-bayashi kan-bayashi left a comment

Choose a reason for hiding this comment

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

Sorry for my mistake! Thanks!

@kan-bayashi kan-bayashi merged commit dd626cc into espnet:master Jul 22, 2023
@Fhrozen Fhrozen deleted the pr-ci-fix branch July 22, 2023 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Travis, Circle CI, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants