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

Update Compression 2021 contributors #2495

Merged
merged 4 commits into from
Nov 16, 2021
Merged

Conversation

rviscomi
Copy link
Member

Progress on #2160

@rviscomi rviscomi added writing Related to wording and content project management Keeping the ball rolling labels Nov 14, 2021
@rviscomi rviscomi added this to the 2021 Launch 🚀 milestone Nov 14, 2021
@rviscomi rviscomi requested a review from lvandeve November 14, 2021 22:24
Comment on lines 5 to 7
authors: [lvandeve]
reviewers: []
analysts: []
Copy link
Member Author

Choose a reason for hiding this comment

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

@lvandeve can you suggest edits to the Authors, Reviewers, and Analysts properties so that they contain everyone who contributed to the chapter, roughly ordered by those with the most significant contributions first? This is what readers will see at the top of the chapter so we want to be sure to give appropriate credit to everyone who helped.

Copy link
Contributor

Choose a reason for hiding this comment

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

Done, I've added the suggestion, there is one more person with significant review I'd like to add whose github username I need to check

Copy link
Contributor

@lvandeve lvandeve Nov 16, 2021

Choose a reason for hiding this comment

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

Added the extra reviewer as well now by editing the original suggestion

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok thanks I'll add them in another PR

Copy link
Member Author

Choose a reason for hiding this comment

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

@rviscomi rviscomi merged commit a5bd92c into main Nov 16, 2021
@rviscomi rviscomi deleted the compression-contributors branch November 16, 2021 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
project management Keeping the ball rolling writing Related to wording and content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants