Skip to content

Refactor markdown2confluence#7

Draft
ottolote wants to merge 19 commits intomainfrom
ol/refactor
Draft

Refactor markdown2confluence#7
ottolote wants to merge 19 commits intomainfrom
ol/refactor

Conversation

@ottolote
Copy link
Contributor

@ottolote ottolote commented May 31, 2024

SUMMARY

Refactor markdown2confluence

STATUS

WIP:

  • Read and respect .confluenceignore file
  • Group pages by repo

CHANGES

Rewrite with focus on:

  • Project decomposition and structure
  • Extensibility through well defined interfaces
  • Loosely coupled testing
  • Use atlassian-python-api package instead of directly accessing deprecated Confluence v1 API

@ottolote ottolote force-pushed the ol/refactor branch 3 times, most recently from e1349a8 to 72e3197 Compare May 31, 2024 12:34
@innofactororg innofactororg deleted a comment from github-actions bot May 31, 2024
@innofactororg innofactororg deleted a comment from github-actions bot May 31, 2024
@ottolote ottolote force-pushed the ol/refactor branch 5 times, most recently from ab3e82d to ad2c2e0 Compare May 31, 2024 13:29
@innofactororg innofactororg deleted a comment from github-actions bot May 31, 2024
@innofactororg innofactororg deleted a comment from github-actions bot May 31, 2024
@innofactororg innofactororg deleted a comment from github-actions bot May 31, 2024
@innofactororg innofactororg deleted a comment from github-actions bot May 31, 2024
@innofactororg innofactororg deleted a comment from github-actions bot May 31, 2024
@github-actions
Copy link

github-actions bot commented Sep 5, 2024

Coverage after merging ol/refactor into main will be

90.27%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
markdown2confluence
   config.py19.05%100%100%19.05%100, 103, 106, 11, 110, 114, 117, 12, 120, 123, 128, 14, 18, 22, 26, 30, 34, 39, 44, 49, 54, 58, 63, 66, 68, 73–75, 77–78, 83, 91, 94, 97
   content_tree.py93.22%100%100%93.22%43, 54, 60, 64
   parser.py93.18%100%100%93.18%16, 28–29
   publisher.py96.88%100%100%96.88%16
   util.py95.24%100%100%95.24%27
tests/e2e
   __init__.py100%100%100%100%
   test_e2e.py100%100%100%100%
tests/integration
   __init__.py100%100%100%100%
   test_integration.py100%100%100%100%
tests/unit
   __init__.py100%100%100%100%
   test_content_tree.py100%100%100%100%
   test_parser.py100%100%100%100%
   test_publisher.py98.53%100%100%98.53%11

@sonarqubecloud
Copy link

Please retry analysis of this Pull-Request directly on SonarCloud

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments