Skip to content

Conversation

dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Oct 21, 2025

v8:x64-windows-static=fail
v8:x64-windows=fail
v8:x86-windows=fail
# https://github.com/vsg-dev/vsgQt/pull/53
Copy link
Member

Choose a reason for hiding this comment

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

This means that it isn't a problem with our build lab and is a problem with the port, so it should probably be a "supports" right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well, I expect upstream to merge the PR and CI to remind us of the changed result when the port is updated.
I dislike the chance of the port being updated with the supports clause preventing test in CI.

Copy link
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

Not worth forcing a rebuild to the nitpick suggestion. But curious about the ci.baseline.txt thing.

Copy link
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

I would prefer supports but it only affects android and I'm happy to leave that optimistic for now...

@BillyONeal BillyONeal enabled auto-merge (squash) October 23, 2025 06:15
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