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.

Sign up for GitHub

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

Jump to bottom

docs: update readme #1475

Merged
merged 2 commits into from
Jan 19, 2024
Merged

docs: update readme #1475

merged 2 commits into from
Jan 19, 2024

Conversation

sajanrajdev
Copy link
Collaborator

Minor updates to the docs

petrovska-petro
petrovska-petro previously approved these changes Jan 3, 2024
View reviewed changes
Copy link
Collaborator

@petrovska-petro petrovska-petro left a comment

Choose a reason for hiding this comment

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

written updates informs better of latest shape of diff msigs and their scope, removed one signer from techops scope ✅

ICEBADGER
ICEBADGER requested changes Jan 3, 2024
View reviewed changes
Copy link

@ICEBADGER ICEBADGER left a comment

Choose a reason for hiding this comment

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

I think there is an extra "s" in line 83 after "required"

Otherwise looks good.

@petrovska-petro petrovska-petro merged commit 17685f5 into main Jan 19, 2024
3 checks passed
@petrovska-petro petrovska-petro deleted the fix-readme branch January 19, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ICEBADGER ICEBADGER ICEBADGER approved these changes

@petrovska-petro petrovska-petro petrovska-petro approved these changes

@rayeaster rayeaster Awaiting requested review from rayeaster

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sajanrajdev @petrovska-petro @ICEBADGER