-
Notifications
You must be signed in to change notification settings - Fork 11
Link to extension and proposed file with requirements #65
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,11 @@ | ||||||
| * Link to the repository: | ||||||
| * License: | ||||||
| * Link to forum threads discussing the extension: | ||||||
| * Description of the extension: input, expected behavior, output | ||||||
| * Compatible OpenRefine version: | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| * Commitment statement: | ||||||
| * Maintenance statement: | ||||||
| * Contact Method: | ||||||
| * AI statement: | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This seems less relevant than a data privacy statement which discloses any data storage, exfiltration, or other special data handling concerns. |
||||||
| * Estimated end-of-life date for the extension: | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Do we need this? Hopefully the answer is "None." This feels kind of like asking parents of a newborn "How long do you expect them to live?" which would be very odd. |
||||||
| * Community achievements: | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This label isn't clear. From looking at the web page, it looks like it's meant to be a citation format or DOI? A clearer label, plus perhaps some placeholder text, would help. |
||||||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -5,10 +5,14 @@ This repository contains a scaffold of an OpenRefine extension, which you can us | |||||||
| See [our guide to writing extensions](https://openrefine.org/docs/technical-reference/writing-extensions) for more information about the process. | ||||||||
|
|
||||||||
| ### Getting started | ||||||||
| Make sure to familiarize yourself with [our extension developer guidelines](https://openrefine.org/docs/technical-reference/extension-dev-guidelines). | ||||||||
|
|
||||||||
| To start your own extension, click the "Use this template" button in the top right corner of this page. | ||||||||
| To start your own extension from this template, click the "Use this template" button in the top right corner of this page. | ||||||||
| This will create a copy of this repository, where you can then change: | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
| * The extension name and description in `module/MOD-INF/module.properties` | ||||||||
| * The `groupId`, `artifactId`, `name` and `description` fields in `pom.xml` | ||||||||
| * Edit this `README.md` file to describe your extension to potential users and contributors instead of the sample extension's own instructions | ||||||||
|
|
||||||||
| ### General Release | ||||||||
|
|
||||||||
| Please ensure that your repository contains the general release file with the relevant information. | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This duplicates the repo license declaration, which we should direct them to update instead.