Skip to content
This repository has been archived by the owner on Apr 5, 2023. It is now read-only.

Add index db wrapper #3

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

OR13
Copy link

@OR13 OR13 commented Dec 7, 2022

Signed-off-by: Orie Steele orie@transmute.industries

OR13 and others added 8 commits January 4, 2023 10:08
Signed-off-by: Orie Steele <orie@transmute.industries>
Signed-off-by: Orie Steele <orie@transmute.industries>
Signed-off-by: Orie Steele <orie@transmute.industries>
Signed-off-by: Orie Steele <orie@transmute.industries>
Signed-off-by: Orie Steele <orie@transmute.industries>
Signed-off-by: Orie Steele <orie@transmute.industries>
Signed-off-by: Orie Steele <orie@transmute.industries>
Signed-off-by: Orie Steele <orie@transmute.industries>
@tkuhrt
Copy link
Collaborator

tkuhrt commented Jan 4, 2023

What is the LICENSE for the Gladstone source files? Can we include a LICENSE file in web-platform-core/Gladstone and possibly SPDX short identifiers in the source files?

@OR13
Copy link
Author

OR13 commented Jan 9, 2023

@tkuhrt it is Apache 2.0, can you suggest the changes you want to see, so I can approve / merge them?

@@ -0,0 +1,81 @@
const gladstone = (database = 'Gladstone') => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
const gladstone = (database = 'Gladstone') => {
// SPDX-License-Identifier: Apache-2.0
const gladstone = (database = 'Gladstone') => {

Copy link
Collaborator

Choose a reason for hiding this comment

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

The above should be good enough. I see that we have a LICENSE file at the root of this repo, which should cover this folder, as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants