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

Add SSWG Update #28

Merged
merged 8 commits into from
Apr 25, 2022
Merged

Add SSWG Update #28

merged 8 commits into from
Apr 25, 2022

Conversation

0xTim
Copy link
Member

@0xTim 0xTim commented Apr 19, 2022

Add's a blog post for the SSWG's Annual Update

0xTim and others added 2 commits April 19, 2022 17:28
Co-authored-by: Mishal Shah <shahmishal@users.noreply.github.com>
@tomerd
Copy link
Contributor

tomerd commented Apr 21, 2022

@tkremenek @cthielen


Since the last update from the SSWG, the Swift on Server ecosystem has continued to grow and expand.

To start, we are very happy to welcome three new members to the SSWG:
Copy link
Member

Choose a reason for hiding this comment

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

The context of "we" here is a bit unclear. Maybe:

To start, the SSWG is happy to welcome three new members:

That removes "we" entirely.

@tkremenek
Copy link
Member

I just did another fresh read. One problem in the post it is keeps saying “we” without saying who “we” is. The post could be altered to not keep saying “we”, or it should clearly define who the “we” is at the start (i.e., the SSWG).

@tomerd tomerd requested a review from cthielen April 21, 2022 17:59
@0xTim
Copy link
Member Author

0xTim commented Apr 21, 2022

@tkremenek OK, updated the post to remove all the first person stuff to make it clear

Co-authored-by: Mishal Shah <shahmishal@users.noreply.github.com>
@0xTim
Copy link
Member Author

0xTim commented Apr 22, 2022

@tkremenek I've added links to everything and addressed the other comments


To start, the SSWG is very happy to welcome three new members:

Adam Fowler
Copy link
Member

Choose a reason for hiding this comment

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

How will these render? In the GitHub preview, these show as just a run of names on the same line. Do we need bullets here?

Copy link
Member

Choose a reason for hiding this comment

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

I'd expect them to render all on one line as written, so bullets would be better.

Copy link
Member Author

Choose a reason for hiding this comment

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

This is a copy paste error, fixed!

@tkremenek
Copy link
Member

@tkremenek I've added links to everything and addressed the other comments

Looks great. I had only one remaining comment about the listing of new SSWG members.

@tomerd
Copy link
Contributor

tomerd commented Apr 22, 2022

@shahmishal do we already have the capability to show a rendered version of the website from a PR?

@0xTim if not, please run locally and attach a screenshot?

@shahmishal
Copy link
Member

shahmishal commented Apr 22, 2022

@shahmishal do we already have the capability to show a rendered version of the website from a PR?

No, this is not supported.

@0xTim
Copy link
Member Author

0xTim commented Apr 23, 2022

Screenshot 2022-04-23 at 09 19 57

@tomerd here you go

@tkremenek
Copy link
Member

This looks great to me. I suggest we wait until after the weekend to post.

@tomerd
Copy link
Contributor

tomerd commented Apr 23, 2022

thanks @0xTim @cthielen and @tkremenek

will merge Monday unless I hear otherwise

---
layout: post
published: true
date: 2022-04-19 11:00:00
Copy link
Member

Choose a reason for hiding this comment

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

This needs to be updated to today's date.

@tomerd
Copy link
Contributor

tomerd commented Apr 25, 2022

@swift-ci test

@tomerd tomerd merged commit ca3b27c into swiftlang:main Apr 25, 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.

5 participants