-
Notifications
You must be signed in to change notification settings - Fork 10
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
README edit and cleanup #40
Conversation
|
| `options?` | `ManifestBuilderOptions` | | ||
|
||
#### Defined in | ||
|
||
[lib/manifestBuilder.ts:41](https://github.com/contentauth/c2pa-node/blob/fb1d732/js-src/lib/manifestBuilder.ts#L41) |
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.
We might need to not do this through a fork, since it seems to be changing the URLs to your fork. Can you push this PR to the main repo, rebuild, and open up a PR from there?
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.
Good catch. Moved to a branch and created new PR.
Close in favor of #41 |
Some small edits, added TOC (which won't be displayed on doc site), made
/docs/README.md
be the API doc index instead of a dup. of the README.Also changed
package.json
to put API doc index indocs/README.md
instead ofdocs/modules.md
.