Skip to content

Conversation

@Kaliumhexacyanoferrat
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor

The following frameworks were updated, pinging maintainers:
genhttp: @Kaliumhexacyanoferrat

@Kaliumhexacyanoferrat Kaliumhexacyanoferrat marked this pull request as ready for review November 20, 2025 16:56
@Kaliumhexacyanoferrat Kaliumhexacyanoferrat changed the title Update GenHTTP to version 10.1 [genhttp] update to version 10.1 Nov 20, 2025
@volyrique
Copy link
Contributor

Did you notice this warning in the test log:

Framework genhttp does not define a default test in benchmark_config.json

@Kaliumhexacyanoferrat
Copy link
Contributor Author

Yes, yet

{
  "framework": "genhttp",  
  "maintainers": ["Kaliumhexacyanoferrat"],
  "tests": [{
    "default": {

@volyrique
Copy link
Contributor

I think that I figured it out - there are too many nested JSON objects in benchmark_config.json; compare with this one, for example.

@Kaliumhexacyanoferrat
Copy link
Contributor Author

Thanks, I will have a look

@Kaliumhexacyanoferrat
Copy link
Contributor Author

Should be fine now, thanks @volyrique

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