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.

Sign up for GitHub

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

Jump to bottom

chore: add Edge & Node to copyright headers #85

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

chore: add Edge & Node to copyright headers #85

wants to merge 1 commit into from

Conversation

hopeyen
Copy link
Collaborator

@hopeyen hopeyen commented Nov 2, 2023

// Copyright 2023-, GraphOps, and Semiotic Labs.
->
// Copyright 2023-, Edge & Node, GraphOps, and Semiotic Labs.

@hopeyen hopeyen added size:small Small p2 Medium priority type:documentation Improvements or additions to documentation labels Nov 2, 2023
@hopeyen hopeyen requested a review from Jannis November 2, 2023 14:56
@hopeyen hopeyen self-assigned this Nov 2, 2023
Jannis
Jannis approved these changes Nov 2, 2023
View reviewed changes
Copy link
Collaborator

@Jannis Jannis left a comment

Choose a reason for hiding this comment

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

Since the order of teams doesn't matter to me, I'm good with this PR. I'd be just as fine with Edge & Node coming last if it matters to GraphOps or Semiotic.

@hopeyen
Copy link
Collaborator Author

hopeyen commented Nov 2, 2023

I figured to do it by alphabetical order because this doesn't matter to me neither 😬
@aasseman lmk what you think, changes are easy to make

aasseman
aasseman approved these changes Nov 2, 2023
View reviewed changes
Copy link
Collaborator

@aasseman aasseman left a comment

Choose a reason for hiding this comment

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

Works for me! Let's wait for The Graph Foundation's recommendations though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jannis Jannis Jannis approved these changes

@aasseman aasseman aasseman approved these changes

Assignees

@hopeyen hopeyen

Labels
p2 Medium priority size:small Small type:documentation Improvements or additions to documentation
Projects
Indexer Experience
Status: ✅ Approved
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@hopeyen @Jannis @aasseman