Skip to content

Added test case for AU_ABN#61

Open
bgbryan2002 wants to merge 9 commits intomainfrom
20-detect-au_abn
Open

Added test case for AU_ABN#61
bgbryan2002 wants to merge 9 commits intomainfrom
20-detect-au_abn

Conversation

@bgbryan2002
Copy link
Copy Markdown

No description provided.

@bgbryan2002 bgbryan2002 added the FutureAssured Team FutureAssured label Mar 22, 2024
@bgbryan2002 bgbryan2002 added this to the Sprint-2 milestone Mar 22, 2024
@bgbryan2002 bgbryan2002 requested review from 1ogcotu and claesmk March 22, 2024 22:44
@bgbryan2002 bgbryan2002 self-assigned this Mar 22, 2024
@bgbryan2002 bgbryan2002 linked an issue Mar 22, 2024 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@1ogcotu 1ogcotu left a comment

Choose a reason for hiding this comment

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

I believe everything look correct, not sure why it is saying you have the checks may have something to due with Pylint.

Copy link
Copy Markdown
Contributor

@claesmk claesmk left a comment

Choose a reason for hiding this comment

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

Looks OK - you just need to address your pylint errors

@claesmk
Copy link
Copy Markdown
Contributor

claesmk commented Mar 25, 2024

@bgbryan2002 reminder you have a 2 pylint errors to resolve

@bgbryan2002
Copy link
Copy Markdown
Author

I reverted the test_au_medicare to its working state but the Pylint and Python app test seem to be recursively buffering the issue. At this point the test are running inconclusively on my end.

@claesmk
Copy link
Copy Markdown
Contributor

claesmk commented Mar 26, 2024

@bgbryan2002 - this should be the reason for you pylint errors you should be OK to merge your PR now

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

Labels

FutureAssured Team FutureAssured

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Detect AU_ABN

3 participants