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

Update README with generated table #64

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

FilipZajdel
Copy link
Contributor

Add a script that generates markdown table based on the contributors index schema. The script outputs the table to the console. The content has to be manually copied into the README.md.

@FilipZajdel FilipZajdel requested a review from greg-fer January 22, 2025 14:48
Copy link
Contributor

@greg-fer greg-fer left a comment

Choose a reason for hiding this comment

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

Not sure about the last "check".

* `releases` are the add-on versions.
| Property | Requirement | Description |
| -------- | ------------ | ----------- |
| name | Required | The name of the application repo. Should be the repo-name in the GitHub URL: https://github.com/org/repo-name.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| name | Required | The name of the application repo. Should be the repo-name in the GitHub URL: https://github.com/org/repo-name.|
| name | Required | The name of the application repository. Should be the `repo-name` in the GitHub URL: https://github.com/org/repo-name.|

| Property | Requirement | Description |
| -------- | ------------ | ----------- |
| name | Required | The name of the application repo. Should be the repo-name in the GitHub URL: https://github.com/org/repo-name.|
| kind | Required | The type of the app repo.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| kind | Required | The type of the app repo.|
| kind | Required | The type of the application repository.|

| kind | Required | The type of the app repo.|
| tags | Required | An array of tags describing the application.|
| releases | Required | The collection of project`s releases.|
| title | Optional | Human readable name of the repo to be shown in the UI. Defaults to the name property.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| title | Optional | Human readable name of the repo to be shown in the UI. Defaults to the name property.|
| title | Optional | Human-readable name of the repository to be shown in the UI. Defaults to the name property.|

| tags | Required | An array of tags describing the application.|
| releases | Required | The collection of project`s releases.|
| title | Optional | Human readable name of the repo to be shown in the UI. Defaults to the name property.|
| description | Optional | Text describing the application. Inferred from the repo if missing.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| description | Optional | Text describing the application. Inferred from the repo if missing.|
| description | Optional | Text describing the application. If missing, it is inferred from the repository.|

| releases | Required | The collection of project`s releases.|
| title | Optional | Human readable name of the repo to be shown in the UI. Defaults to the name property.|
| description | Optional | Text describing the application. Inferred from the repo if missing.|
| manifest | Optional | Alternative filename for the west manifest. Defaults to west.yml.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| manifest | Optional | Alternative filename for the west manifest. Defaults to west.yml.|
| manifest | Optional | Alternative file name for the west manifest. Defaults to `west.yml`.|

| title | Optional | Human readable name of the repo to be shown in the UI. Defaults to the name property.|
| description | Optional | Text describing the application. Inferred from the repo if missing.|
| manifest | Optional | Alternative filename for the west manifest. Defaults to west.yml.|
| license | Optional | The name of the application license, e.g. "Apache 2.0". Inferred from the repo if missing.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| license | Optional | The name of the application license, e.g. "Apache 2.0". Inferred from the repo if missing.|
| license | Optional | The name of the application license, for example "Apache 2.0". If missing, it is inferred from the repository.|

| description | Optional | Text describing the application. Inferred from the repo if missing.|
| manifest | Optional | Alternative filename for the west manifest. Defaults to west.yml.|
| license | Optional | The name of the application license, e.g. "Apache 2.0". Inferred from the repo if missing.|
| apps | Optional | Glob pattern to find directories containing applications.Applications need a *.conf file and a CMakeLists.txt file at their root. The glob expressions are used to match directories, so no file pattern is necessary.By default, the VS Code extension will assume that there's just a single application sitting at the root of the repo.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| apps | Optional | Glob pattern to find directories containing applications.Applications need a *.conf file and a CMakeLists.txt file at their root. The glob expressions are used to match directories, so no file pattern is necessary.By default, the VS Code extension will assume that there's just a single application sitting at the root of the repo.|
| apps | Optional | Glob pattern to find directories containing applications. Applications need a `*.conf` file and the `CMakeLists.txt` file at their root. The glob expressions are used to match directories, so no file pattern is necessary. By default, the nRF Connect for Visual Studio Code extension will assume that there is just a single application sitting at the root of the repository.|

| manifest | Optional | Alternative filename for the west manifest. Defaults to west.yml.|
| license | Optional | The name of the application license, e.g. "Apache 2.0". Inferred from the repo if missing.|
| apps | Optional | Glob pattern to find directories containing applications.Applications need a *.conf file and a CMakeLists.txt file at their root. The glob expressions are used to match directories, so no file pattern is necessary.By default, the VS Code extension will assume that there's just a single application sitting at the root of the repo.|
| defaultBranch | Optional | The default git branch that the repository is checked out. Inferred from the repo if missing.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| defaultBranch | Optional | The default git branch that the repository is checked out. Inferred from the repo if missing.|
| defaultBranch | Optional | The default git branch that the repository is checked out at. If missing, it is inferred from the repository.|

| license | Optional | The name of the application license, e.g. "Apache 2.0". Inferred from the repo if missing.|
| apps | Optional | Glob pattern to find directories containing applications.Applications need a *.conf file and a CMakeLists.txt file at their root. The glob expressions are used to match directories, so no file pattern is necessary.By default, the VS Code extension will assume that there's just a single application sitting at the root of the repo.|
| defaultBranch | Optional | The default git branch that the repository is checked out. Inferred from the repo if missing.|
| docsUrl | Optional | The URL of the add-on's documentation|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| docsUrl | Optional | The URL of the add-on's documentation|
| docsUrl | Optional | The URL of the add-on's documentation.|

| apps | Optional | Glob pattern to find directories containing applications.Applications need a *.conf file and a CMakeLists.txt file at their root. The glob expressions are used to match directories, so no file pattern is necessary.By default, the VS Code extension will assume that there's just a single application sitting at the root of the repo.|
| defaultBranch | Optional | The default git branch that the repository is checked out. Inferred from the repo if missing.|
| docsUrl | Optional | The URL of the add-on's documentation|
| restricted | Optional | Mark the restricted access to any of the dependencies.|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| restricted | Optional | Mark the restricted access to any of the dependencies.|
| restricted | Optional | Check for marking the restricted access to any of the dependencies.|

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.

2 participants