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] Links in static HTML files should be relative #4236

Merged
merged 1 commit into from
May 9, 2024

Conversation

bruntib
Copy link
Contributor

@bruntib bruntib commented May 3, 2024

The links in the static HTML files generated by "CodeChecker parse -e html" should be relative, so the output folder of the above command is portable.

@bruntib bruntib added bugfix 🔨 tools 🛠️ Meta-tag for all the additional tools supplied with CodeChecker: plist2html, tu_collector, etc. plist2html 🌏 report-converter 🔄 labels May 3, 2024
@bruntib bruntib added this to the release 6.24.0 milestone May 3, 2024
@bruntib bruntib requested review from noraz31 and cservakt May 3, 2024 15:10
@bruntib bruntib requested a review from vodorok as a code owner May 3, 2024 15:10
# There are 3 reports in the test file.
self.assertEqual(len(re.findall('"link": "', content)), 3)
# The links should be relative so the static HTML folder is
# prtable.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# prtable.
# portable.

The links in the static HTML files generated by "CodeChecker parse -e html"
should be relative, so the output folder of the above command is portable.
@bruntib bruntib force-pushed the static_html_relative_link branch from f233107 to 2aa4fdb Compare May 9, 2024 12:32
Copy link
Collaborator

@vodorok vodorok left a comment

Choose a reason for hiding this comment

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

LGTM!

@vodorok vodorok merged commit 8d1513a into Ericsson:master May 9, 2024
7 of 8 checks passed
@bruntib bruntib deleted the static_html_relative_link branch May 15, 2024 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix 🔨 plist2html 🌏 report-converter 🔄 tools 🛠️ Meta-tag for all the additional tools supplied with CodeChecker: plist2html, tu_collector, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants