Skip to content

Conversation

@tdcox
Copy link
Contributor

@tdcox tdcox commented Dec 1, 2022

Signed-off-by: Terry Cox terry@bootstrap.je

Changes

  • Migrate narrative documents to cdevents.dev repo
  • Add navigation links to README.md

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

Copy link
Member

@afrittoli afrittoli left a comment

Choose a reason for hiding this comment

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

Thank you for this.

cdevents.drawio was not migrated to the website repo so it should not be removed.

@tdcox
Copy link
Contributor Author

tdcox commented Dec 2, 2022

cdevents.drawio was not migrated to the website repo so it should not be removed.

It is in this commit: cdevents/cdevents.dev@864e3c9

The GitHub website can't display a folder tree in which a submodule has been removed and replaced with a real folder.

@afrittoli
Copy link
Member

cdevents.drawio was not migrated to the website repo so it should not be removed.

It is in this commit: cdevents/cdevents.dev@864e3c9

The GitHub website can't display a folder tree in which a submodule has been removed and replaced with a real folder.

Ah, thank you, I missed it 🙏

Signed-off-by: Terry Cox <terry@bootstrap.je>
Copy link
Member

@afrittoli afrittoli left a comment

Choose a reason for hiding this comment

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

Thank you for the updates, lgtm.

@e-backmark-ericsson wdyt?

Copy link
Contributor

@e-backmark-ericsson e-backmark-ericsson left a comment

Choose a reason for hiding this comment

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

It looks ok to me. It's maybe a bit strange to have a roadmap managed through a website repo, but it is good enough for now at least.

@e-backmark-ericsson
Copy link
Contributor

This PR was approved some days ago. Any reason for not merging it?

@afrittoli afrittoli merged commit 03ca228 into cdevents:main Dec 13, 2022
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.

3 participants