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.

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

Second website review #192

Merged
merged 18 commits into from
Oct 4, 2021
Merged

Second website review #192

merged 18 commits into from
Oct 4, 2021

Conversation

DanieldeR
Copy link
Contributor

A bunch of changes based on the review of the website:

  • updating favicon
  • minor copy edits on the homepage
  • enabling the community section
  • fixing the high-level diagram broken link
  • linking to the published blog post

And assorted fixes throughout the repo:

  • Updated the readme to point at the blog post
  • Updating the order in contributors.md
  • Updating security.md
  • Adding new governance.md

@netlify
Copy link

netlify bot commented Oct 1, 2021

✔️ Deploy Preview for elated-stonebraker-105904 ready!

🔨 Explore the source changes: cf6c90e

🔍 Inspect the deploy log: https://app.netlify.com/sites/elated-stonebraker-105904/deploys/615b7a2af0f5fa00070c2449

😎 Browse the preview: https://deploy-preview-192--elated-stonebraker-105904.netlify.app/

@squeedee squeedee self-requested a review October 4, 2021 15:33
CONTRIBUTING.md Outdated Show resolved Hide resolved
Copy link
Member

@squeedee squeedee left a comment

Choose a reason for hiding this comment

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

Should probably fix the .T typo, the rest is fine for now, just nitpicky comments.

site/content/posts/announcing-cartographer.md Outdated Show resolved Hide resolved
site/content/posts/announcing-cartographer.md Outdated Show resolved Hide resolved
@DanieldeR DanieldeR merged commit 49c8fe5 into main Oct 4, 2021
@DanieldeR DanieldeR deleted the second-website-review branch October 4, 2021 22:06
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