Skip to content
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.

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

fix: allow one click access for html-report (#286) #287

Merged
merged 2 commits into from
Jan 7, 2022

Conversation

ahmetgeymen
Copy link
Contributor

@ahmetgeymen ahmetgeymen commented Dec 28, 2021

Removing space character from timestamp based path to allow one click (cmd+click) access on the terminal of vs-code on macOS.

issue: #286

removing space character from timestamp based path to allow one click (cmd+click) access on the terminal of vs-code on macOS.

Signed-off-by: Ahmet Geymen <ahmetgeymen@outlook.com>
@sriv sriv added the ReleaseCandidate Add this label to a PR to trigger the relase action for HTML Report when ever this PR is merged. label Dec 29, 2021
@gaugebot
Copy link

gaugebot bot commented Dec 29, 2021

@ahmetgeymen Thank you for contributing to html-report. 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.

@zabil
Copy link
Member

zabil commented Dec 29, 2021

@ahmetgeymen you can refer baa4f41 for reference on how to bump up versions.

@ahmetgeymen
Copy link
Contributor Author

ahmetgeymen commented Dec 29, 2021

@zabil Actually latest release was tagged as v4.1.2 so I didn't think to bump up for 4.1.4. I thought this change might go with 4.1.3. I would bump up for 4.1.4 as you wish.

Signed-off-by: Ahmet Geymen <ahmetgeymen@outlook.com>
@ahmetgeymen
Copy link
Contributor Author

Is there anything special to do for macOS and windows environment. It seems tests are failing for those OSs.

@sriv sriv merged commit f1a095e into getgauge:master Jan 7, 2022
@sriv
Copy link
Member

sriv commented Jan 7, 2022

I ran the tests locally on windows and macos, they passed for me. So I guess these are transient failures - rerun passed confirming this.

Thanks for the pull request @ahmetgeymen !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ReleaseCandidate Add this label to a PR to trigger the relase action for HTML Report when ever this PR is merged.
Development

Successfully merging this pull request may close these issues.

None yet

3 participants