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

Add ORT configurations and instructions #24

Merged
merged 2 commits into from
Jun 19, 2024
Merged

Conversation

OliverStolzBO
Copy link
Collaborator

  • Add .ort.yml to fix mistakes made by the ORT scanner
  • Add release note instructions

- Add .ort.yml to fix mistakes made by the ORT scanner
- Add release note instructions
README.md Outdated
@@ -232,6 +232,18 @@ The program crashes and it has something to do with the Scheduler the extension
> To display errors and to generate the code visualization, the extension calls the [Scheduler](https://github.com/iml130/pfdl). The Scheduler is still under development, so there may be bugs. If the extension crashes because of the Scheduler please open an issue in that repo.

## Release Notes
For each release, a `.vsix` installation file for VS Code is provided, as well as a `bundle.js` source code file, containing the code that is responsible for the code visualization.

Additionally, a license file is generated using the [OSS Review Toolkit (ORT)](http://oss-review-toolkit.org/ort/) to automatically conclude licenses and copyright information declared in the dependencies of this project (excluding `devDependencies` for npm). The file can be reproduced by following the [ORT installation guide](http://oss-review-toolkit.org/ort/docs/getting-started/installation) and running the following commands from inside the downloaded ORT repository:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it conclude or include?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that conclude is not the correct word here, changed it to "collect and summarize"

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright LFTM now :)

@maxhoerstr maxhoerstr merged commit da292f7 into main Jun 19, 2024
3 checks passed
@OliverStolzBO OliverStolzBO deleted the ort-configuration branch July 8, 2024 15:50
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