From 462933fe16543ba392745222aeb220db88cfe403 Mon Sep 17 00:00:00 2001 From: yohanelly95 Date: Wed, 2 Aug 2023 10:30:54 +0530 Subject: [PATCH 1/4] chore: contribution guidelines --- CONTRIBUTING.md | 12 ++++++++++++ 1 file changed, 12 insertions(+) create mode 100644 CONTRIBUTING.md diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md new file mode 100644 index 0000000..f03c154 --- /dev/null +++ b/CONTRIBUTING.md @@ -0,0 +1,12 @@ + +# Contributing to Razor Network Datasources +If you wish to propose a new price feed to be added to the datasources used by the protocol, please follow the steps below: + +1. Start by forking the repository and creating your branch from master. +2. Add the new job to `jobs.json` and update `collections.json` as needed with the correct jobIDs. +3. Execute the test suite using the command `npm run test`. It's crucial to ensure all tests pass. Additionally, please confirm that your code is properly formatted and aligns with the existing style guide. +4. Issue a pull request (PR) to the upstream master branch. +5. The proposed changes will be available for community discussion and review in GitHub discussions. Community members are welcome to provide feedback and votes on the proposal. + + + From 4f3048c236bd3ff6c59f6d65c7bf0f436519007d Mon Sep 17 00:00:00 2001 From: yohanelly95 Date: Thu, 19 Sep 2024 12:38:40 +0530 Subject: [PATCH 2/4] chore: fix runner version --- .github/workflows/ci.yml | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 3e3cc54..8f964a4 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -3,9 +3,9 @@ name: CI on: workflow_dispatch: push: - branches: [ "master" ] + branches: ["master"] pull_request_target: - branches: [ "master" ] + branches: ["master"] jobs: tests: @@ -13,20 +13,28 @@ jobs: strategy: matrix: - node-version: [16.x] + node-version: [20.x] # Updated to a more recent LTS version steps: - - uses: actions/checkout@v3 + - name: Checkout code + uses: actions/checkout@v3 with: ref: ${{ github.event.pull_request.head.sha }} + - name: Check branch run: git branch + - name: Use Node.js ${{ matrix.node-version }} uses: actions/setup-node@v3 with: node-version: ${{ matrix.node-version }} cache: "npm" + - name: Install Dependencies - run: npm install + run: | + npm --version + node --version + npm install + - name: Run Tests - run: npm test + run: npm run test From 6a3f00bd2244cd43a46d6c3d02633a268f83edbb Mon Sep 17 00:00:00 2001 From: yohanelly95 Date: Thu, 19 Sep 2024 12:56:56 +0530 Subject: [PATCH 3/4] chore: run ci on merge commit to master target rather than the forked repos specific commit --- .github/workflows/ci.yml | 5 ----- 1 file changed, 5 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 8f964a4..69f89ec 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -18,11 +18,6 @@ jobs: steps: - name: Checkout code uses: actions/checkout@v3 - with: - ref: ${{ github.event.pull_request.head.sha }} - - - name: Check branch - run: git branch - name: Use Node.js ${{ matrix.node-version }} uses: actions/setup-node@v3 From 334e803316dd8fe57027811cc8d708b9a35ad423 Mon Sep 17 00:00:00 2001 From: yohanelly95 Date: Thu, 19 Sep 2024 13:12:01 +0530 Subject: [PATCH 4/4] chore: trigger on pull_request --- .github/workflows/ci.yml | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 69f89ec..c1f5e9c 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -4,25 +4,23 @@ on: workflow_dispatch: push: branches: ["master"] - pull_request_target: + pull_request: branches: ["master"] jobs: tests: runs-on: self-hosted - strategy: - matrix: - node-version: [20.x] # Updated to a more recent LTS version - steps: - name: Checkout code uses: actions/checkout@v3 + with: + ref: ${{ github.event.pull_request.head.sha }} - - name: Use Node.js ${{ matrix.node-version }} + - name: Use Node.js uses: actions/setup-node@v3 with: - node-version: ${{ matrix.node-version }} + node-version: 20 cache: "npm" - name: Install Dependencies