website icon indicating copy to clipboard operation
website copied to clipboard

Update tooling-data.yaml by adding 2 tools

Open zx80 opened this issue 4 months ago • 12 comments

What kind of change does this PR introduce?

It adds to tools which are related to JSON Schema: a set of JSON Schema utils and a compiler which can export type description to JSON Schema.

Issue Number:

Closes #1831.

Screenshots/videos:

NA.

If relevant, did you update the documentation?

NA.

Summary

It adds new tools in the long list of available tools.

It is a re-submission of #1636 that I thought had been merged, but I was wrong. I have removed the mention of the next version as it seemed to generate some concerns.

The combination of the two tools constitute a viable and rather fast JSON Schema compiler by using the schema-to-model converter of the first tool and the model compiler from the second tool.

Does this PR introduce a breaking change?

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

zx80 avatar Aug 26 '25 06:08 zx80

Hi @zx80! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: issue reference

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

github-actions[bot] avatar Aug 26 '25 06:08 github-actions[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 100.00%. Comparing base (dcf4ea4) to head (40c6477).

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1830   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           30        30           
  Lines          633       633           
  Branches       196       196           
=========================================
  Hits           633       633           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Aug 26 '25 06:08 codecov[bot]

Hi @zx80! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: issue reference

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

github-actions[bot] avatar Aug 26 '25 07:08 github-actions[bot]

Hi @zx80! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: issue reference

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Hello bot.

ISTM that there is an issue reference (#1831) in the PR description.

zx80 avatar Aug 26 '25 08:08 zx80

Hi @zx80! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: issue reference

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

github-actions[bot] avatar Aug 26 '25 09:08 github-actions[bot]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
website ✅ Ready (View Log) Visit Preview 40c6477d6c3b05c9b27eb6306c4936cc1adb483d

github-actions[bot] avatar Aug 26 '25 09:08 github-actions[bot]

As I do not understand how to please the CI bots, I submitted these two entries through the issue form instead, see #1832 et #1833.

zx80 avatar Aug 26 '25 09:08 zx80

@zx80 There's some documentation on how to connect a PR to an Issue here: https://docs.github.com/en/issues/tracking-your-work-with-issues/using-issues/linking-a-pull-request-to-an-issue

Relequestual avatar Nov 08 '25 11:11 Relequestual

@Relequestual: I closed this issue to use the form submission path instead. Should I reopen this one with keywords added to please de CI, or are the form submission sufficient and will be processed at some point in the future?

zx80 avatar Nov 09 '25 06:11 zx80

@zx80 Once the GH Issue is done and agreed, you should re-open the PR or create a new one, your choice. The form is just a proxy for creating an Issue. Thanks.

Relequestual avatar Nov 22 '25 15:11 Relequestual

Hi @zx80! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: issue reference

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

github-actions[bot] avatar Nov 22 '25 16:11 github-actions[bot]

@zx80 Once the GH Issue is done and agreed, you should re-open the PR or create a new one, your choice. The form is just a proxy for creating an Issue. Thanks.

Ok… It seems that the check wanted a magic word before the issue reference which was already there. The automatic message is not really helpful. Sigh.

zx80 avatar Nov 22 '25 16:11 zx80