Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix file path handling for reports on Windows #742

Merged
merged 3 commits into from
Sep 22, 2021

Conversation

at6ue
Copy link
Contributor

@at6ue at6ue commented Sep 21, 2021

The file path of the report was handled wrongly as in #737.
Nothing has happened when Windows users execute "Show Last Run Report" command or click on the status bar.

getReportPath is expected to return only a file path generated by html-report plugin, as far as I have found.

Only tested manually on macOS Big Sur and Windows 10.

Signed-off-by: Atsuo INOUE <at6ue@outlook.jp>
@gaugebot
Copy link

gaugebot bot commented Sep 21, 2021

@at6ue Thank you for contributing to gauge-vscode. Your pull request has been labeled as a release candidate 🎉🎉.

Merging this PR will trigger a release.

Please bump up the version as part of this PR.

Instructions to bump the version can found at CONTRIBUTING.md

If the CONTRIBUTING.md file does not exist or does not include instructions about bumping up the version, please looks previous commits in git history to see what changes need to be done.

@sriv
Copy link
Member

sriv commented Sep 21, 2021

Good catch @at6ue . Marked this as an RC, please bump the version. Thanks.

at6ue and others added 2 commits September 22, 2021 09:58
Signed-off-by: Atsuo INOUE <at6ue@outlook.jp>
@sriv sriv merged commit a7e3ce9 into getgauge:master Sep 22, 2021
@at6ue at6ue deleted the fix_openReport branch September 22, 2021 17:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Development

Successfully merging this pull request may close these issues.

2 participants