Skip to content

Module add#3

Open
rse2019coder wants to merge 7 commits into
masterfrom
module_add
Open

Module add#3
rse2019coder wants to merge 7 commits into
masterfrom
module_add

Conversation

@rse2019coder

Copy link
Copy Markdown
Collaborator

Merge request completing tasks described by Issue #2

Changes

  • Modularised calculation code
  • Added unit tests for new my_function.py module
  • Automated unit tests using Azure Pipelines

Comments

The scope of the Issue does not cover adding additional functionality on top of the addition code, so I have limited myself to simply adding the framework for future arithmetical functions.

@rse2019coder rse2019coder added the enhancement New feature or request label Sep 15, 2019

@adcwilliams adcwilliams left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Automated tests don't run successfully! I'm getting:

src/test_my_function.py:9: AssertionError

Comment thread azure-pipelines.yml
displayName: 'Run a one-line script'
- task: UsePythonVersion@0
inputs:
versionSpec: '3.7'

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it work in version 3.7?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants