From f1d250c70bd751967aca349c70842ffccdf61bd2 Mon Sep 17 00:00:00 2001
From: Ardian <34317628+arddluma@users.noreply.github.com>
Date: Fri, 16 Sep 2022 13:29:49 +0200
Subject: [PATCH] Pipeline improvements (#109)

* Pipeline improvements

* Change mumbai RPC

* Downgrade node
---
 .circleci/config.yml                       |  6 +++--
 .github/workflows/run-linter-and-tests.yml |  3 +++
 extensions/constants.ts                    |  2 +-
 pull_request_template.md                   | 26 ++++++++++++++++++++++
 4 files changed, 34 insertions(+), 3 deletions(-)
 create mode 100644 pull_request_template.md

diff --git a/.circleci/config.yml b/.circleci/config.yml
index aaa1a0fd..d85d52fc 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -7,7 +7,7 @@ jobs:
   install:
     working_directory: ~/etherspot-contracts
     docker:
-      - image: circleci/node:14.17.0
+      - image: cimg/node:14.17.0
         auth:
           username: $DOCKERHUB_USER
           password: $DOCKERHUB_PASSWORD
@@ -33,7 +33,7 @@ jobs:
   publish:
     working_directory: ~/etherspot-contracts
     docker:
-      - image: circleci/node:14.17.0
+      - image: cimg/node:14.17.0
         auth:
           username: $DOCKERHUB_USER
           password: $DOCKERHUB_PASSWORD
@@ -88,6 +88,8 @@ workflows:
                   - master
       - publish-github-release:
           context: general-vars
+          requires:
+            - publish
           filters:
             branches:
               only:
diff --git a/.github/workflows/run-linter-and-tests.yml b/.github/workflows/run-linter-and-tests.yml
index 8d268b4f..ffd50be3 100644
--- a/.github/workflows/run-linter-and-tests.yml
+++ b/.github/workflows/run-linter-and-tests.yml
@@ -44,3 +44,6 @@ jobs:
 
       - name: Run tests
         run: npm run test -- --report-gas
+      
+      - name: Dry run build
+        run: npm run build
\ No newline at end of file
diff --git a/extensions/constants.ts b/extensions/constants.ts
index 20060966..d5bfd28c 100644
--- a/extensions/constants.ts
+++ b/extensions/constants.ts
@@ -122,7 +122,7 @@ export const NETWORK_CONFIGS: {
   },
   [NetworkNames.Mumbai]: {
     chainId: 80001,
-    defaultProviderUrl: "https://rpc-mumbai.maticvigil.com",
+    defaultProviderUrl: "https://matic-mumbai.chainstacklabs.com",
     defaultGasPrice: 5,
     explorer: "https://mumbai.polygonscan.com",
   },
diff --git a/pull_request_template.md b/pull_request_template.md
new file mode 100644
index 00000000..2e058eba
--- /dev/null
+++ b/pull_request_template.md
@@ -0,0 +1,26 @@
+<!--- Provide a general summary of your changes in the Title above -->
+
+## Description
+<!--- Describe your changes in detail -->
+-
+
+## Motivation and Context
+<!--- Why is this change required? What problem does it solve? -->
+<!--- If it fixes an open issue, please link to the issue here. -->
+-
+
+## How Has This Been Tested?
+<!--- Please describe in detail how you tested your changes. -->
+<!--- Include details of your testing environment, and the tests you ran to -->
+<!--- see how your change affects other areas of the code, etc. -->
+-
+-
+-
+
+## Screenshots (if appropriate):
+
+## Types of changes
+<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->
+- [ ] Bug fix (non-breaking change which fixes an issue)
+- [ ] New feature (non-breaking change which adds functionality)
+- [ ] Breaking change (fix or feature that would cause existing functionality to change)
\ No newline at end of file