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

Patch 2 #492

Closed
wants to merge 9 commits into from
Closed

Conversation

BitBlocks368
Copy link

Spelling and grammar changes

rickymac68 and others added 9 commits March 27, 2023 17:59
This is the initial commit for the governance security CIP based on discussions at the Longmont Voltaire CIP-1694 Working Group.
correct the format for spacing
Added the correct authors
Posted the pull request discussion link to the discussion section of the header.
Correct typos in the abstract

Co-authored-by: Adam Dean <63186174+Crypto2099@users.noreply.github.com>
Added threat number 1 to the Specification section to begin the process.
Added an initial list of governance threats for public review, comment, input, and feedback.
Brackets not needed. N/A until responses are received.

Co-authored-by: Adam Dean <63186174+Crypto2099@users.noreply.github.com>
Spelling and grammar changes
@Ryun1
Copy link
Collaborator

Ryun1 commented Mar 30, 2023

Hi @BitBlocks368,

I believe you wish to contribute to @rickymac68's CPS-???? | Governance Security, opening a pull request here is the incorrect method to collaborate.

Instead of opening a pull request on this repository you could:

  • Leave comments with amendments on Rick's CPS PR for him to make updates.
    OR
  • Fork Rick's fork of CIPs, from here, then make your changes. Following this make a pull request from your fork of Rick's into Rick's fork (rather than a pull request into this repository). Rick can then merge in changes to the CPS.

For now I will close this PR.

@Ryun1 Ryun1 closed this Mar 30, 2023
@BitBlocks368 BitBlocks368 deleted the patch-2 branch October 12, 2023 04:21
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.

3 participants