Skip to content

Sewerpipe#4

Merged
oocube merged 16 commits into
mainfrom
sewerpipe
Jun 10, 2026
Merged

Sewerpipe#4
oocube merged 16 commits into
mainfrom
sewerpipe

Conversation

@oocube

@oocube oocube commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Index all items, even those that might get overridden.

@mcarans

mcarans commented Jun 8, 2026

Copy link
Copy Markdown

@oocube If I understand correctly, the end output of this is generated HTML. Is that right? If so, is there anywhere I can view the HTML generated by this PR? I tried looking at the output of the builds but they seem to be the Java jar files not the HTML.

@oocube

oocube commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator Author

There are unit tests that get executed during the maven build. Unless they pass there is no jar file.
But you are right, the html output is generated only when the code executes. I did these tests locally, and they also run as part of https://github.com/OoliteProject/oolite-expansion-catalog (check the referenced website).

If you want to see a sample run I'd have to revive the 'scan' workflow in this project. At least the release step is no longer appropriate.

@oocube

oocube commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator Author

I added a test run which indexes 20 expansions. The output is available as an additional build artifact. Please take a look.

@mcarans

mcarans commented Jun 9, 2026

Copy link
Copy Markdown

I added a test run which indexes 20 expansions. The output is available as an additional build artifact. Please take a look.

The html looks great, but the release here seems to only have the sources: https://github.com/OoliteProject/OoliteAddonScanner/releases. Previous releases had 4 artifacts.

@oocube

oocube commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator Author

The artifacts are back. Thank you for spotting this.

@mcarans mcarans left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me!

@oocube oocube merged commit 073e8de into main Jun 10, 2026
8 checks passed
@oocube oocube deleted the sewerpipe branch June 10, 2026 05:13
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.

2 participants