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.

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

fix: Use https:// URLs in report templates #5582

Merged
merged 1 commit into from
Mar 23, 2023
Merged

fix: Use https:// URLs in report templates #5582

merged 1 commit into from
Mar 23, 2023

Conversation

Marcono1234
Copy link
Contributor

@Marcono1234 Marcono1234 commented Mar 22, 2023 •
edited
Loading

Fixes Issue

none

Description of Change

  • Update URLs in report templates to use https:// instead of http://
  • Update HTML and Jenkins report template to also convert https:// license URLs to links

Have test cases been added to cover the new functionality?

no

@boring-cyborg boring-cyborg bot added the core changes to core label Mar 22, 2023
@Marcono1234 Marcono1234 changed the title Use https:// URLs in report templates fix: Use https:// URLs in report templates Mar 22, 2023
@jeremylong jeremylong added this to the 8.2.1 milestone Mar 23, 2023
@jeremylong jeremylong merged commit 32aa38f into jeremylong:main Mar 23, 2023
@jeremylong
Copy link
Owner

Thanks for the PR!

@Marcono1234 Marcono1234 deleted the report-templates-https-urls branch March 23, 2023 19:35
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jeremylong jeremylong jeremylong approved these changes

Assignees
No one assigned
Labels
core changes to core
Projects
None yet
Milestone
8.2.1
Development

Successfully merging this pull request may close these issues.
2 participants
@Marcono1234 @jeremylong