Skip to content

fix: test link in html reporter#5224

Merged
JoshuaKGoldberg merged 3 commits intomochajs:mainfrom
danny0838:main
Oct 29, 2024
Merged

fix: test link in html reporter#5224
JoshuaKGoldberg merged 3 commits intomochajs:mainfrom
danny0838:main

Conversation

@danny0838
Copy link
Copy Markdown
Contributor

PR Checklist

Overview

Fix as mentioned in the related issue comment.

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Oct 11, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: dependabot[bot] (708e5f0)
  • ✅ login: JoshuaKGoldberg / name: Josh Goldberg ✨ (473e7d8)
  • ✅ login: danny0838 / name: Danny Lin (6e8bcdf)

Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! Just one style nit, but nothing functionally blocking. 🚀

Will leave open for a bit in case anybody from @mochajs/maintenance-crew wants to take a look.

Comment thread lib/reporters/html.js Outdated
@JoshuaKGoldberg
Copy link
Copy Markdown
Member

Oh, and - please sign the CLA so we can accept this @danny0838.

@danny0838 danny0838 force-pushed the main branch 2 times, most recently from 6ec2434 to 92af340 Compare October 12, 2024 00:58
dependabot Bot and others added 2 commits October 14, 2024 17:11
Bumps the github-actions group with 1 update: [joelwmale/webhook-action](https://github.com/joelwmale/webhook-action).


Updates `joelwmale/webhook-action` from 2.3.2 to 2.4.1
- [Release notes](https://github.com/joelwmale/webhook-action/releases)
- [Commits](joelwmale/webhook-action@2.3.2...2.4.1)

---
updated-dependencies:
- dependency-name: joelwmale/webhook-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
- `grep` may not work as expected if a previous `fgrep` or `invert` parameter exists.
@JoshuaKGoldberg JoshuaKGoldberg changed the title fix test link in html reporter fix: test link in html reporter Oct 19, 2024
Comment thread .github/workflows/nightly-site-deploy.yml Outdated
@JoshuaKGoldberg JoshuaKGoldberg merged commit f054acc into mochajs:main Oct 29, 2024
lennonnikolas pushed a commit to lennonnikolas/mocha that referenced this pull request Jan 24, 2026
* build(deps): bump the github-actions group with 1 update

Bumps the github-actions group with 1 update: [joelwmale/webhook-action](https://github.com/joelwmale/webhook-action).


Updates `joelwmale/webhook-action` from 2.3.2 to 2.4.1
- [Release notes](https://github.com/joelwmale/webhook-action/releases)
- [Commits](joelwmale/webhook-action@2.3.2...2.4.1)

---
updated-dependencies:
- dependency-name: joelwmale/webhook-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: remove bad test link params in HTML reporter

- `grep` may not work as expected if a previous `fgrep` or `invert` parameter exists.

* Update .github/workflows/nightly-site-deploy.yml

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Josh Goldberg ✨ <git@joshuakgoldberg.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 Bug: test links in the browser reporter keep the original fgrep and invert params

2 participants