Create pylint.yml

.github/workflows/pylint.yml

@@ -0,0 +1,23 @@
+name: Pylint
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.8", "3.9", "3.10"]
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v2
+      with:
+        python-version: ${{ matrix.python-version }}
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install pylint
+    - name: Analysing the code with pylint
+      run: |
+        pylint $(git ls-files '*.py')

app.py

@@ -1,12 +1,11 @@
 #!/usr/bin/env python
-"""Flask server to serve the block API."""
 
-import json
 from time import time
-import argparse
-from uuid import uuid4
 from flask import Flask, request
+import json
+from uuid import uuid4
 from blockchain import Blockchain, Transaction, Block
+import argparse
 
 parser = argparse.ArgumentParser(description="A blockchain simulation")
 parser.add_argument("-p", "--port", type=int, help="Port number", default=80)
@@ -16,19 +15,18 @@ port = args.port
 
 app = Flask(__name__)
 
-NODE_IDENTIFIER = str(uuid4()).replace("-", "")
+node_identifier = str(uuid4()).replace("-", "")
 
 blockchain = Blockchain()
 
 
 @app.route("/mine", methods=["GET"])
 def mine():
-    """Mine samy coin."""
     last_block = blockchain.blocks[-1]
     last_proof = last_block.proof
     proof = blockchain.proof_of_work(last_proof)
 
-    transaction = Transaction("Network", NODE_IDENTIFIER, 50)
+    transaction = Transaction("Network", node_identifier, 50)
     blockchain.new_transaction(transaction)
     block = Block(
         len(blockchain.blocks) + 1,
@@ -44,7 +42,6 @@ def mine():
 
 @app.route("/transactions/new", methods=["POST"])
 def new_transaction():
-    """Submit a new transaction."""
     values = request.get_json()
     transaction = Transaction(values["sender"], values["recipient"], values["amount"])
     blockchain.new_transaction(transaction)
@@ -54,29 +51,27 @@ def new_transaction():
 
 @app.route("/chain", methods=["GET"])
 def chain():
-    """Display the block chain."""
     return json.dumps([b.__dict__ for b in blockchain.blocks])
 
 
 @app.route("/nodes", methods=["GET"])
 def nodes():
-    """List known nodes."""
     return json.dumps(list(blockchain.nodes))
 
 
 @app.route("/register", methods=["POST"])
 def register_nodes():
-    """Register a new node."""
-    json_values = request.get_json()
-    for node in json_values["nodes"]:
+    values = request.get_json()
+    nodes = values["nodes"]
+
+    for node in nodes:
         blockchain.register_node(node)
 
     return "New node has been added\n"
 
 
 @app.route("/resolve", methods=["GET"])
-def resolve():
-    """Resolve the blockchain."""
+def consensus():
     replaced = blockchain.resolve_conflicts()
 
     if replaced: