Skip to content

adding unique id to fault categories - #42

Merged
arcuri82 merged 4 commits into
masterfrom
adding-ids
Oct 2, 2026
Merged

arcuri82 merged 4 commits into
masterfrom
adding-ids

Conversation

@arcuri82

@arcuri82 arcuri82 commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@arcuri82
arcuri82 requested a review from Stranger6667 October 1, 2026 07:46

@Stranger6667 Stranger6667 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It would be nice to add this new field to the report schema too, otherwise looks good to me :)

@arcuri82

arcuri82 commented Oct 2, 2026

Copy link
Copy Markdown
Contributor Author

@Stranger6667 sounds good. I ll do it in a different PR

@arcuri82
arcuri82 merged commit 18ea8fe into master Oct 2, 2026
4 checks passed
@arcuri82
arcuri82 deleted the adding-ids branch October 2, 2026 12:15
@arcuri82

arcuri82 commented Oct 2, 2026

Copy link
Copy Markdown
Contributor Author

@Stranger6667 btw, would you have any use if those schema files were published in a PyPi package, automatically updated at each new WFC release? right now I m just publishing them for Java on Maven Central. but can (relatively easily) add them to PyPi as well if needed

@Stranger6667

Copy link
Copy Markdown
Collaborator

@arcuri82

Thanks! Yes, that would be useful.

Right now I keep a hand-copied report schema in the tests, and it drifts with every WFC release. A PyPI package would let me validate the reports against the real upstream schema as a dev dependency.

I'd rather not add a runtime dependency, so the auth schema will stay vendored for now :)

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