Skip to content

test: check that system.filePath() generates file extensions #765

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Apr 3, 2022
Merged

test: check that system.filePath() generates file extensions #765

merged 3 commits into from
Apr 3, 2022

Conversation

ST-DDT
Copy link
Member

@ST-DDT ST-DDT commented Apr 3, 2022

Ref: #541

@ST-DDT ST-DDT added the c: test label Apr 3, 2022
@ST-DDT ST-DDT requested review from a team April 3, 2022 11:30
@ST-DDT ST-DDT self-assigned this Apr 3, 2022
@ST-DDT ST-DDT added p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug labels Apr 3, 2022
ST-DDT
ST-DDT commented Apr 3, 2022
View reviewed changes
@ST-DDT ST-DDT mentioned this pull request Apr 3, 2022
Closed
@codecov
Copy link

codecov bot commented Apr 3, 2022 •
edited
Loading

Codecov Report

Merging #765 (1cd88af) into main (b40121e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #765   +/-   ##
=======================================
  Coverage   99.34%   99.34%           
=======================================
  Files        1922     1922           
  Lines      176505   176505           
  Branches      909      909           
=======================================
  Hits       175352   175352           
  Misses       1097     1097           
  Partials       56       56

Shinigami92
Shinigami92 reviewed Apr 3, 2022
View reviewed changes
@Shinigami92 Shinigami92 requested a review from a team April 3, 2022 13:02
@ST-DDT ST-DDT merged commit 52ad16e into main Apr 3, 2022
@ST-DDT ST-DDT deleted the tests/system/filePath-file-extension branch April 3, 2022 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shinigami92 Shinigami92 Shinigami92 approved these changes

@ejcheng ejcheng ejcheng approved these changes

Assignees

@ST-DDT ST-DDT

Labels
c: test p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug
Projects
None yet
Milestone
v6.1 - First bugfixes
Development

Successfully merging this pull request may close these issues.
3 participants
@ST-DDT @Shinigami92 @ejcheng