Skip to content

Lauching ZEP website#2

Merged
sanketverma1704 merged 2 commits into
zarr-developers:mainfrom
sanketverma1704:main
May 28, 2022
Merged

Lauching ZEP website#2
sanketverma1704 merged 2 commits into
zarr-developers:mainfrom
sanketverma1704:main

Conversation

@sanketverma1704

Copy link
Copy Markdown
Member

Hi. 🙋🏻‍♂️

This PR creates a website for ZEPs at https://zarr.dev/zeps. All the incoming ZEPs will be displayed on the web pages to facilitate easy reviewing and general reading.

Thanks! 🖖🏻

@sanketverma1704 sanketverma1704 merged commit a9ad838 into zarr-developers:main May 28, 2022
@@ -0,0 +1,29 @@
---

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This file isn't needed, eh?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes! I'll remove it. This file is created whenever someone creates a new Jekyll project.

@joshmoore

Copy link
Copy Markdown
Member

NB: I'm not overly sure about having images copied multiple places (assets/images/flowchart.png, and the logo) We might want to just hotlink. I mention because when the logo changed, I had to go looking for copies of the old one. People start using them in their presentations, and it kind of leads to a mess.

see: https://github.com/zarr-developers/community/tree/main/logos

@sanketverma1704

Copy link
Copy Markdown
Member Author

NB: I'm not overly sure about having images copied multiple places (assets/images/flowchart.png, and the logo) We might want to just hotlink. I mention because when the logo changed, I had to go looking for copies of the old one. People start using them in their presentations, and it kind of leads to a mess.

see: https://github.com/zarr-developers/community/tree/main/logos

Makes sense. I'll link the logo from GitHub then.

joshmoore pushed a commit that referenced this pull request Jul 5, 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.

2 participants