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

Remove actions that require write access on pull request #389

Merged
merged 5 commits into from
Oct 22, 2023
Merged

Remove actions that require write access on pull request #389

merged 5 commits into from
Oct 22, 2023

Conversation

MSevey
Copy link
Member

@MSevey MSevey commented Oct 22, 2023

Overview

This is a further simplification of the CI by removing all write actions on the PR so that we can revert back to the pull_request trigger.

liamsi
liamsi reviewed Oct 22, 2023
View reviewed changes
Copy link
Member

@liamsi liamsi 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 this is a good idea!

@MSevey MSevey marked this pull request as ready for review October 22, 2023 18:46
@MSevey MSevey enabled auto-merge October 22, 2023 18:46
@liamsi liamsi requested review from adlerjohn, YazzyYaz, Bidon15, a team and rach-id October 22, 2023 20:03
auto-merge was automatically disabled October 22, 2023 20:12

Merge queue setting changed

@liamsi liamsi merged commit a1981e6 into celestiaorg:master Oct 22, 2023
trader-payne added a commit to trader-payne/networks that referenced this pull request Oct 24, 2023
@trader-payne @Bidon15
@tty47 @evan-forbes @YazzyYaz @mindstyle85 @itrocket-team @jcstein @MSevey @smuu @rootulp @kinrokinro @AviaOne @AlexToTheMoon @liamsi @ilhanu @packetstracer @kj89 @mabalaru @larry0x @neoprom @Brightlystake @suntzu93 @GPvalidator @quantumcore1 @rooiie @EdouardLvdl @staking4all @daniel-ck89 @qubelabsio @redref @llPorZall @y3v63n @matilote @ivandiazperez @Gunter038 @adlerjohn @shivlim @travellingsoldier85 @curlycrypto184 @DasRasyo @stakemepro @easy2stake @nodebreaker0-0 @okannako @jmc000 @richard-stakingcabin @cryptovestor21
fast forward (#1)
bb8cdfb 
* Update genesis.json

* feat: add mocha-2 genesis

* feat: add mocha-2 genesis (celestiaorg#326)

* fix: final fixed genesis (celestiaorg#327)

* feat: initial genesis after hardspoon of arabica-8 for arabica-9 (celestiaorg#328)

* feat: initial genesis after hardspoon of arabica-8 for arabica-9

* chore: updated genesis file

* fix: comply with removing genesis_time in minter module

* chore: finalise the genesis.json for arabica-9 chain (celestiaorg#329)

* feat!: update mocha-3 genesis

* chore: add first hash

* fix: add port

* Create seeds.txt

* Add itrocket peer

* fix: add arabica-9 peer

* feat: add genesis for arabica-10

Signed-off-by: Smuu <[email protected]>

* feat: get hardspoon file from vali

Signed-off-by: Smuu <[email protected]>

* feat: add genesis for arabica-10

Signed-off-by: Smuu <[email protected]>

* feat(arabica-10): update the genesis file

Signed-off-by: Jose Ramon Mañes <[email protected]>

* feat: add arabica-10 peers.txt (celestiaorg#343)

* feat: add arabica-10 peers.txt

* Add more peers

Untested

* feat: add mocha-4 genesis (celestiaorg#344)

* Update README.md with version matrix (celestiaorg#345)

* Update README.md with version matrix

* Update README.md to public link

* Create seeds.txt

* Nodes.Guru mocha-4 peers (celestiaorg#347)

* Update seeds.txt

* itrocket mocha-4 peer and seed (celestiaorg#353)

* Add itrocket peer

* itrocket mocha-4 seed

* Update seeds.txt (celestiaorg#352)

Add "AM Solutions" seed

Co-authored-by: Nguyen Nhu Viet <[email protected]>

* feat(ci): initialize validate genesis ci

feat(ci): initialize validation of validator PR user

* update CI from test repo

* add check for gentest file exist

* remove unneeded file

* add celestia network pre-genesis.json

* add gentx dir

* Create peers.txt

* add gentx Staker Space

* add check for gentx directory exist

* testing fork checkout

* debuggin

* testing pull_request_target

* testing

* Add Blockscope.net gentx

* Add Blockscope peer

* Add kjnodes gentx

* Create ScalaVentures.json gentx

* bug: update constants used in check to be magnitude of utia not tia

* update comment and error

* add larry gentx

* add larry peer

* add PAT for commiting to the csv file

* Delete celestia/gentx/matt.json

* fix: simplify CI (celestiaorg#380)

- remove tracking of PRs in usersprs.csv
- remove corresponding go code, csv file, and CI job

* Create Neoprom-gentx.json

* Add gentx and peers for Brightlystake

* Add Suntzu.json

* GPvalidator.json with Gentx folder

* Create quantumcore.json

* Swiss Staking AG gentx

* Gentx P-OPS Team

* Add amsolutions-gentx.json

* updated jq gentx

* chore: simplify the 80 percent sheet

* Add Stakin gentx

* Add Stakin peers.txt

* POPS Update peers.txt

* staking4all gentx

* staking4all gentx

* Update Peers

* Remove actions that require write access on pull request (celestiaorg#389)

* Remove actions that require write access on pull request

* fix char

* fix copy paste error

* add logging

* Add Nodes.Guru gentx

* Update peers.txt

add staker-space peer to list

* fix bash

* test remove brackets

* testing empty json

* update for valid gentx

* remove json

* Add Keplr gentx

* Update Keplr's peer

* fix: print formatted file correctly

* Gentx Qubelabs

* Update peers.txt

* Gentx Kiln

* fix: attempt to fix CI issues by using a specific version of go and celestia-app

* Gentx ContributionDAO

* fix: intall error

* fix: add silviugae

* fix: grasping at CI straws

* Update/Add amsolutions-gentx.json

* Update/Add amsolutions-gentx.json

* Update amsolutions-gentx.json

* Update gentx-ContributionDAO.json

* Update gentx-ContributionDAO.json

* Apply suggestions from code review

Co-authored-by: Ismail Khoffi <[email protected]>

* Update gentx-ContributionDAO.json

* Update gentx-ContributionDAO.json

* Update gentx-ContributionDAO.json

* Update gentx-ContributionDAO.json

* Create gentx-moonli_me.json

* Update peers.txt

* Update peers.txt

* Add Twinstake gentx

* BloClick

* jq Twinstake.json

* Create Gunter.json

* fix: sheet in utia for validation to work and fix peers.txt whil at it (celestiaorg#446)

* Update gentx-ContributionDAO.json

* Add Twinstake peers.txt

* Added 4sv peer (celestiaorg#424)

Co-authored-by: Ismail Khoffi <[email protected]>

* kooltek68 gentx and peer for celestia chain (celestiaorg#419)

* Add kooltek68 gentx

* Add kooltek68 peer

---------

Co-authored-by: Ismail Khoffi <[email protected]>
Co-authored-by: John Adler <[email protected]>

* Added StakeWithUs gentx (celestiaorg#402)

* Added StakeWithUs gentx

* Added StakeWithUs peer

---------

Co-authored-by: Poh YH <[email protected]>
Co-authored-by: Yaz Khoury <[email protected]>
Co-authored-by: Ismail Khoffi <[email protected]>
Co-authored-by: John Adler <[email protected]>

* Add Conqueror gentx & peer (celestiaorg#434)

* Create Conqueror.json

* Update peers.txt

---------

Co-authored-by: Ismail Khoffi <[email protected]>

* Update validate-validator-github.yml (celestiaorg#460)

* Add gentx STAKEME (celestiaorg#462)

* Create gentx STAKEME.json

* Create peer STAKEME.txt

---------

Co-authored-by: Ismail Khoffi <[email protected]>

* Adding Easy2Stake GENTX and peer information (celestiaorg#442)

* Adding Easy2Stake genesis file

* Update peers.txt with Easy2Stake info

* Update and rename peers.txt to easy2stake.txt

* Revert "Update and rename peers.txt to easy2stake.txt"

This reverts commit 961c4d0.

* Revert "Update peers.txt with Easy2Stake info"

This reverts commit 4c47744.

* Create easy2stake.txt

---------

Co-authored-by: Alex <[email protected]>
Co-authored-by: Ismail Khoffi <[email protected]>
Co-authored-by: John Adler <[email protected]>

* Add B-Harvest gentx (celestiaorg#461)

* Add B-Harvest gentx

* Update B-Harvest.json

* Gentx & Peer NakoTurk  (celestiaorg#436)

* Create NakoTurk.json

* Create peers.txt

* Update NakoTurk.json

* Update NakoTurk.json

* Add files via upload

* Update NakoTurk.json

* Update peers.txt

* Rename peers.txt to NakoTurk.txt

* Create NakoTurk.txt

* Delete celestia/gentx/NakoTurk.txt

---------

Co-authored-by: Ismail Khoffi <[email protected]>

* add Node Guardians gentx (celestiaorg#423)

* add StakingCabin gentx and peer (celestiaorg#464)

* add StakingCabin gentx and peer

* add StakingCabin gentx and peer

* Wavefive gentx and peers (celestiaorg#465)

* Create wavefive.txt

* Add files via upload

---------

Signed-off-by: Smuu <[email protected]>
Signed-off-by: Jose Ramon Mañes <[email protected]>
Co-authored-by: Nguyen Nhu Viet <[email protected]>
Co-authored-by: Jose Ramon Mañes <[email protected]>
Co-authored-by: evan-forbes <[email protected]>
Co-authored-by: Evan Forbes <[email protected]>
Co-authored-by: Yaz Khoury <[email protected]>
Co-authored-by: mindstyle85 <[email protected]>
Co-authored-by: Marutyan Samvel <[email protected]>
Co-authored-by: joshcs.eth ᵍᵐ <[email protected]>
Co-authored-by: Matthew Sevey <[email protected]>
Co-authored-by: Smuu <[email protected]>
Co-authored-by: Jose Ramon Mañes <[email protected]>
Co-authored-by: Rootul P <[email protected]>
Co-authored-by: 0xAN <[email protected]>
Co-authored-by: Aviaone <[email protected]>
Co-authored-by: Alexey <[email protected]>
Co-authored-by: Ismail Khoffi <[email protected]>
Co-authored-by: “ilhanu” <[email protected]>
Co-authored-by: packettsracer <[email protected]>
Co-authored-by: kjnodes <[email protected]>
Co-authored-by: mabalaru <[email protected]>
Co-authored-by: larry <[email protected]>
Co-authored-by: neoprom <[email protected]>
Co-authored-by: Brightlystake <[email protected]>
Co-authored-by: Suntzu <[email protected]>
Co-authored-by: GPvalidator <[email protected]>
Co-authored-by: quantumcore1 <[email protected]>
Co-authored-by: creeea <[email protected]>
Co-authored-by: rooiie <[email protected]>
Co-authored-by: Edouard Lavidalle <[email protected]>
Co-authored-by: staking4all <[email protected]>
Co-authored-by: Ilhan <[email protected]>
Co-authored-by: Daniel <[email protected]>
Co-authored-by: Qubelabs <[email protected]>
Co-authored-by: Anthony BESCOND <[email protected]>
Co-authored-by: Pongchai Tangbowonweerakun <[email protected]>
Co-authored-by: y3v63n <[email protected]>
Co-authored-by: Mateusz Jędrzejewski <[email protected]>
Co-authored-by: ivandiazperez <[email protected]>
Co-authored-by: Gunter <[email protected]>
Co-authored-by: John Adler <[email protected]>
Co-authored-by: shivlim <[email protected]>
Co-authored-by: kooltek68 <[email protected]>
Co-authored-by: curlycrypto184 <[email protected]>
Co-authored-by: Poh YH <[email protected]>
Co-authored-by: Conqueror <[email protected]>
Co-authored-by: STAKEME <[email protected]>
Co-authored-by: easy2stake <[email protected]>
Co-authored-by: Alex <[email protected]>
Co-authored-by: JeseonLEE <[email protected]>
Co-authored-by: NakoTurk <[email protected]>
Co-authored-by: jmc000 <[email protected]>
Co-authored-by: richard@stakingcabin <[email protected]>
Co-authored-by: Jim | wavefive <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liamsi liamsi liamsi approved these changes

@adlerjohn adlerjohn Awaiting requested review from adlerjohn

@YazzyYaz YazzyYaz Awaiting requested review from YazzyYaz

@Bidon15 Bidon15 Awaiting requested review from Bidon15

@rach-id rach-id Awaiting requested review from rach-id

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

Successfully merging this pull request may close these issues.
2 participants
@MSevey @liamsi