Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial JSON Schema for license report tool #1713

Closed
wants to merge 5 commits into from
Closed

Initial JSON Schema for license report tool #1713

wants to merge 5 commits into from

Conversation

pubmikeb
Copy link
Contributor

@pubmikeb pubmikeb commented Jul 6, 2021

No description provided.

@pubmikeb
Copy link
Contributor Author

pubmikeb commented Jul 6, 2021

@madskristensen, I've prepared an initial JSON Schema for license report tool.
However, the build #1713 is failed although the schema itself is valid.

I've had a look on test results but no clear indication what's wrong.

@madskristensen
Copy link
Contributor

@GerryFerdinandus Is this due to the missing grunt task?

@GerryFerdinandus
Copy link
Contributor

@pubmikeb
Build server log:
Warning: Schema file name must be present in the catalog URL. (see: src/api/json/catalog.json)=> license-report-config.json Use --force to continue.

See contributing
You must also add it to the catalog.

@pubmikeb
Copy link
Contributor Author

pubmikeb commented Jul 7, 2021

@GerryFerdinandus , thanks a lot for pointing me to this issue.

I've added a data to the catalog:

{
	"name": "license-report-config.json",
	"description": "JSON Schema for license report tool configuration file",
	"fileMatch": [
		"license-report-config.json"
	],
	"url": "https://json.schemastore.org/license-report-config.json"
},

But the commit #1715 of this change to the catalog fails the test. The only info I see:

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/runner/.npm/_logs/2021-07-07T06_13_06_840Z-debug.log
Error: Process completed with exit code 3.

Do I have an access to the build server log to be able to better understand the reason of the fail?

@pubmikeb
Copy link
Contributor Author

pubmikeb commented Jul 7, 2021

Replaced by consolidated #1716

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.

3 participants