Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

oss-review-toolkit/ort

renovate[bot] opened a pull request on oss-review-toolkit/ort
deps: update ksp to v2.0.21-1.0.26
This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.devtools.ksp:symbol-processing-api](https://goo.gle/ksp...
renovate[bot] created a branch on oss-review-toolkit/ort

renovate/ksp - A suite of tools to automate software compliance checks.

sschuberth pushed 4 commits to main oss-review-toolkit/ort
  • docs(model): Improve `LicenseFilePatterns` docs Add class docs and slightly reword property docs, also reducing sent... 786aba4
  • refactor(model): Make `LicenseFilePatterns` properties sets This better reflects that duplicates make no sense. Sig... 6c7a4b1
  • refactor(model)!: Rename a `LicenseFilePatterns` property `rootLicenseFilenames` sounded too much as if it would tak... d1fa585
  • chore(model)!: Remove old plugin config aliases The old names were deprecated more than a year ago, so it should be ... e4e8396

View on GitHub

sschuberth deleted a branch oss-review-toolkit/ort

lfp-imps

sschuberth closed a pull request on oss-review-toolkit/ort
Some `LicenseFilePatterns` improvements
Please have a look at the individual commit messages for the details.
fviernau created a review on a pull request on oss-review-toolkit/ort

View on GitHub

sschuberth created a review comment on a pull request on oss-review-toolkit/ort
> The JSON returned is not in 'package.json' format, wich is what I've been trying to say. Now I see what you mean... I believe this is something that git broken in ad9a3639604aacfa6aa3fd8315932...

View on GitHub

sschuberth created a review on a pull request on oss-review-toolkit/ort

View on GitHub

fviernau created a review comment on a pull request on oss-review-toolkit/ort
Do you still think the PR needs to be re-worked? How shall we proceed ?

View on GitHub

fviernau created a review on a pull request on oss-review-toolkit/ort

View on GitHub

sschuberth created a review comment on a pull request on oss-review-toolkit/ort
> This is what I've been assuming as well when starting with this PR, until I came across this `requireSuccess()` call here, which left me surprised: Sorry, I mixed things up: `run()` is a short...

View on GitHub

sschuberth created a review on a pull request on oss-review-toolkit/ort

View on GitHub

sschuberth pushed 2 commits to purl-imps oss-review-toolkit/ort
  • fix(model): Rework purl conversion according to the specs Implement the pseudo-algorithm described at [1]. Most impo... 5b6e616
  • test(model): Test against the official purl test suite data The data has been copied from [1]. [1]: https://github.... f2dca01

View on GitHub

sschuberth pushed 1 commit to lfp-imps oss-review-toolkit/ort
  • chore(model)!: Remove old plugin config aliases The old names were deprecated more than a year ago, so it should be ... 6622f9f

View on GitHub

sschuberth pushed 1 commit to lfp-imps oss-review-toolkit/ort
  • refactor(model)!: Rename a `LicenseFilePatterns` property `rootLicenseFilenames` sounded too much as if it would tak... d297309

View on GitHub

sschuberth created a review comment on a pull request on oss-review-toolkit/ort
> Does the de-serialization of the config fail in case the alias is omit? Yes. > If yes (it does not go unnoticed), shall we just make this a breaking change to avoid having this historical t...

View on GitHub

sschuberth created a review on a pull request on oss-review-toolkit/ort

View on GitHub

github-advanced-security[bot] created a review comment on a pull request on oss-review-toolkit/ort
## Reports code blocks that are not followed by an empty line Missing empty line after block. [Show more details](https://github.com/oss-review-toolkit/ort/security/code-scanning/3974)

View on GitHub

github-advanced-security[bot] created a review comment on a pull request on oss-review-toolkit/ort
## Reports files that do not follow ORT's order for imports Imports in file '/home/runner/work/ort/ort/model/src/main/kotlin/utils/PurlUtils.kt' are not sorted alphabetically or single blank lines...

View on GitHub

github-advanced-security[bot] created a review on a pull request on oss-review-toolkit/ort

View on GitHub

sschuberth pushed 2 commits to purl-imps oss-review-toolkit/ort
  • fix(model): Rework purl conversion according to the specs Implement the pseudo-algorithm described at [1]. Most impo... 625aece
  • test(model): Test against the official purl test suite data The data has been copied from [1]. [1]: https://github.... 15705d4

View on GitHub

sschuberth created a review comment on a pull request on oss-review-toolkit/ort
Discarding an intermediate ".." indeed does not make any sense, so I went for normalizing as it at least doe snot break the test suite.

View on GitHub

sschuberth created a review on a pull request on oss-review-toolkit/ort

View on GitHub

sschuberth created a review comment on a pull request on oss-review-toolkit/ort
Let's keep it now that I've changed it.

View on GitHub

sschuberth created a review on a pull request on oss-review-toolkit/ort

View on GitHub

sschuberth created a review comment on a pull request on oss-review-toolkit/ort
I'd prefer to stick to `KNOWN_QUALIFIER_KEYS`. It's shorter and does not hard-code any implications into its name.

View on GitHub

sschuberth created a review on a pull request on oss-review-toolkit/ort

View on GitHub

fviernau created a review comment on a pull request on oss-review-toolkit/ort
Sorry for the typo in `KNOW_QUALIFIER_KEYS`, it made it into the diff.

View on GitHub

fviernau created a review on a pull request on oss-review-toolkit/ort

View on GitHub

fviernau created a comment on a pull request on oss-review-toolkit/ort
> Wouldn't we get test failures then? In expected output like files yet, in file which are used only as input probably no.

View on GitHub

Load more