-
-
Notifications
You must be signed in to change notification settings - Fork 63
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: arduino/setup-protoc
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: c65c819
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: arduino/setup-protoc
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 3ea1d70
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 5 commits
- 3 files changed
- 2 contributors
Commits on Jan 31, 2024
-
Install referenced schema in "npm:validate" task
The "npm:validate" task validates the repository's `package.json` npm manifest file against its JSON schema to catch any problems with its data format. In order to avoid duplication of content, JSON schemas may reference other schemas via the `$ref` keyword. The `package.json` schema was recently updated to share resources with the "base" configuration schema, which caused the validation to start failing: schema /tmp/package-json-schema-norSGPxlCR.json is invalid error: can't resolve reference https://json.schemastore.org/base.json#/definitions/license from id https://json.schemastore.org/package.json# task: Failed to run task "npm:validate": exit status 1 The solution is to configure the task to download that schema as well and also to provide its path to the avj-cli validator via a `-r` flag.
Configuration menu - View commit details
-
Copy full SHA for 0dd9ef8 - Browse repository at this point
Copy the full SHA 0dd9ef8View commit details -
Merge pull request #96 from per1234/install-referenced-schema
Install referenced schema in "npm:validate" task
Configuration menu - View commit details
-
Copy full SHA for a3ad94c - Browse repository at this point
Copy the full SHA a3ad94cView commit details
Commits on Mar 20, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 91b36bb - Browse repository at this point
Copy the full SHA 91b36bbView commit details
Commits on Sep 3, 2024
-
Configure actions/upload-artifact action to upload required hidden files
A breaking change was made in the 3.2.1 release of the "actions/upload-artifact" action, without doing a major version bump as would be done in a responsibly maintained project. The action now defaults to not uploading "hidden" files. This project's "Check npm Dependencies" workflow uses the "Licensed" tool to check for incompatible dependency licenses. The dependencies license metadata cache used by Licensed is stored in a folder named `.licensed`. In order to facilitate updates, the workflow uploads the generated dependencies license metadata cache as a workflow artifact when the current cache is found to be outdated. The `.` at the start of the `.licensed` folder name causes it to now not be uploaded to the workflow artifact. In order to catch such problems, the workflow configures the "actions/upload-artifact" action to fail if no files were uploaded. So in addition to not uploading the artifact, the change in the "actions/upload-artifact" action's behavior also resulted in the workflow runs failing: Error: No files were found with the provided path: .licenses/. No artifacts will be uploaded. The problem is fixed by disabling the "actions/upload-artifact" action's new behavior via the `include-hidden-files` input. After this change, the workflow can once more upload the dependencies license metadata cache to a workflow artifact as needed.
Configuration menu - View commit details
-
Copy full SHA for 436c800 - Browse repository at this point
Copy the full SHA 436c800View commit details -
Merge pull request #103 from per1234/include-hidden-files
Configure actions/upload-artifact action to upload required hidden files
Configuration menu - View commit details
-
Copy full SHA for 3ea1d70 - Browse repository at this point
Copy the full SHA 3ea1d70View commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff c65c819...3ea1d70